tags 293340 + patch pending
thanks

* Martin Michlmayr <[EMAIL PROTECTED]> [2005-02-02 16:13]:
> There is an inconsistency with the values reported for boolean
> variables.  It says "False" for false but "1" for true values.  It
> should use the same scheme for both:
> 
>   --vbr, -v         generate variable bitrate files [False] +
> vs
>   --vbr, -v         generate variable bitrate files [1] +
> 
> In fact, given that jack accepts "yes"/"no" and not 1/0 or True/False,
> it should probably say "yes"/"no".

Actually, here's a really simple fix for this:

diff -urN jack-3.1.1~/jack_utils.py jack-3.1.1/jack_utils.py
--- jack-3.1.1~/jack_utils.py   2005-06-19 19:43:29.001768184 +0100
+++ jack-3.1.1/jack_utils.py    2005-06-19 19:52:14.788836400 +0100
@@ -135,7 +135,7 @@
             s = "yes"
         else:
             s = "no"
-    if what['type'] == types.StringType:
+    elif what['type'] == types.StringType:
         s = "'%s'" % what['val']
     else:
         s = str(what['val'])

The first if (checking for 'toggle') is completely ignored because of
the second if - it will simply use the else branch.
-- 
Martin Michlmayr
http://www.cyrius.com/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to