Code review comment for lp:~trb143/openlp/ThemeManager

Revision history for this message
Michael Gorven (mgorven) wrote :

+ log.debug(r.x(), r.y(), r.width(),r.height())
+ log.debug(self._theme.BackgroundParameter2)
Needs a format string.

+# log.debug(u" "Just split where you can"
+# log.debug(u" "Getting the words split right"
+# log.debug(u" "This is how they split", lines
+# log.debug(u" "Now render them"
+# log.debug(u" "Line %2d: Render '%s' at (%d, %d) wh=(%d,%d)"%(
linenum, line, x, y,w,h)
I know these are commented out, but if it's code the syntax should still be
correct.

+ print bbox
+ print "AA"
+ print tree.find('BackgroundType')
+ print "AAA"
Should use log.debug().

 review approve

review: Approve

« Back to merge proposal