openlp-core team mailing list archive

[Michael Gorven <michael@xxxxxxxxxxxxx>]

Review: Approve
+            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

-- 
https://code.launchpad.net/~trb143/openlp/ThemeManager/+merge/5014
Your team openlp.org Core is subscribed to branch lp:openlp.