[feedgnuplot] 07/13: minor simplification

Dima Kogan dima at secretsauce.net
Sun Oct 16 03:55:02 UTC 2016


This is an automated email from the git hooks/post-receive script.

dkogan-guest pushed a commit to branch debian
in repository feedgnuplot.

commit 5123ca73d3eda279260b14420f56c4823e28700d
Author: Dima Kogan <dima at secretsauce.net>
Date:   Sat Oct 15 20:14:33 2016 -0700

    minor simplification
    
    I delete the style and styleall options after I use them. This is defensive and
    clarifies the intent
---
 bin/feedgnuplot | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/bin/feedgnuplot b/bin/feedgnuplot
index 940ae42..020707b 100755
--- a/bin/feedgnuplot
+++ b/bin/feedgnuplot
@@ -138,17 +138,13 @@ sub interpretCommandline
   # --curvestyleall, so fill that in
   if( $options{styleall} )
   {
-    if($options{curvestyleall} )
-    {
-      $options{curvestyleall} .= " $options{styleall}";
-    }
-    else
-    {
-      $options{curvestyleall} = $options{styleall};
-    }
+    if($options{curvestyleall} ) { $options{curvestyleall} .= " $options{styleall}"; }
+    else                         { $options{curvestyleall}  =   $options{styleall};  }
+    delete $options{styleall};
   }
-  push @{$options{curvestyle}}, @{$options{style}};
 
+  push @{$options{curvestyle}}, @{$options{style}};
+  delete $options{style};
 
   # --legend and --curvestyle options are conceptually hashes, but are parsed as
   # arrays in order to preserve the ordering. I parse both of these into hashes

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-science/packages/feedgnuplot.git



More information about the debian-science-commits mailing list