Re: Improve HTML output of regression tests (issue 5342042)

[adam . spiers]

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/