Pushed to staging as 1294535b
http://codereview.appspot.com/5342042/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
passes make and no reg test diffs.
james
http://codereview.appspot.com/5342042/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
New patch set uploaded.
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py
File scripts/build/output-distance.py (right):
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py#newcode8
scripts/build/output-distance.py:8: from cgi import escape
Only stylistic comments:
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py
File scripts/build/output-distance.py (right):
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py#newcode8
scripts/build/output-distance.py:8: from cgi import escape
On 2011/11/07 09:15:04, dak wrote:
Uh, yes? See line 1313: the argument list consists of an arbitrary
number of _pairs_.
Right - it's comparing (baseline, check) pairs. I didn't introduce the
modulo, I just made it explicit which is more legible than relying on
Python's interpretation of 0 as
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py
File scripts/build/output-distance.py (right):
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py#newcode1304
scripts/build/output-distance.py:1304: if len (args) % 2 == 1:
On 2011/11/07 08:5
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py
File scripts/build/output-distance.py (right):
http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py#newcode1304
scripts/build/output-distance.py:1304: if len (args) % 2 == 1:
why is there a mod
passes make and no reg test diffs.
James
http://codereview.appspot.com/5342042/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel