[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: convert-ly (issue 2670) (issue 6610058)
From: |
lemzwerg |
Subject: |
Re: convert-ly (issue 2670) (issue 6610058) |
Date: |
Sat, 06 Oct 2012 05:06:56 +0000 |
http://codereview.appspot.com/6610058/diff/2001/scripts/convert-ly.py
File scripts/convert-ly.py (right):
http://codereview.appspot.com/6610058/diff/2001/scripts/convert-ly.py#newcode357
scripts/convert-ly.py:357: sys.exit(errors)
Are you going to report the number of errors using the `errors'
variable? In case this is true, I would consider this a bad idea, since
you abuse the functionality of the exit status.
How large can `errors' become? The value returned by `exit' must be in
the range 0-255 (with 0 meaning `no error').
http://codereview.appspot.com/6610058/
- convert-ly (issue 2670) (issue 6610058), julien . rioux, 2012/10/05
- Re: convert-ly (issue 2670) (issue 6610058),
lemzwerg <=
- Re: convert-ly (issue 2670) (issue 6610058), lemzwerg, 2012/10/06
- Re: convert-ly (issue 2670) (issue 6610058), janek . lilypond, 2012/10/08
- Re: convert-ly (issue 2670) (issue 6610058), dak, 2012/10/08
- Re: convert-ly (issue 2670) (issue 6610058), graham, 2012/10/08
- Re: convert-ly (issue 2670) (issue 6610058), dak, 2012/10/08