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 On 2011/11/07 15:53:49, Julien Rioux wrote:
Since the style within lilypond's python sources is to import the
module into
the namespace, it would be nice to keep to that style, e.g. "import
cgi" here,
and then...
Done. http://codereview.appspot.com/5342042/diff/1/scripts/build/output-distance.py#newcode462 scripts/build/output-distance.py:462: str = '<font size="-2"><pre>%s</pre></font>' % escape(str) On 2011/11/07 15:53:49, Julien Rioux wrote:
...use "cgi.escape (str)" here. The space before the parenthesis is
also to keep
the consistent style.
Done. http://codereview.appspot.com/5342042/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
