openlp-core team mailing list archive

[Andreas Preikschat <andreaspreikschat@xxxxxxxxxx>]

Andreas Preikschat has proposed merging lp:~googol/openlp/render into lp:openlp.

Requested reviews:
  Tim Bentley (trb143)
Related bugs:
  Bug #856296 in OpenLP: "Improve performance of song loading for Preview/Live"
  https://bugs.launchpad.net/openlp/+bug/856296

For more details, see:
https://code.launchpad.net/~googol/openlp/render/+merge/77037

Hello,

I have improved the line counting.
-- 
https://code.launchpad.net/~googol/openlp/render/+merge/77037
Your team OpenLP Core is subscribed to branch lp:openlp.

=== modified file 'openlp/core/lib/renderer.py'
--- openlp/core/lib/renderer.py	2011-09-26 16:57:32 +0000
+++ openlp/core/lib/renderer.py	2011-09-26 18:50:30 +0000
@@ -44,6 +44,7 @@
     'Get those children out of the muddy, muddy \n' \
     '{r}C{/r}{b}h{/b}{bl}i{/bl}{y}l{/y}{g}d{/g}{pk}' \
     'r{/pk}{o}e{/o}{pp}n{/pp} of the Lord\n'
+VERSE_FOR_LINE_COUNT = u'\n'.join(map(unicode, xrange(50)))
 FOOTER = [u'Arky Arky (Unknown)', u'Public Domain', u'CCLI 123456']
 
 class Renderer(object):
@@ -190,7 +191,7 @@
         serviceItem.theme = theme_data
         if self.force_page:
             # make big page for theme edit dialog to get line count
-            serviceItem.add_from_text(u'', VERSE + VERSE + VERSE)
+            serviceItem.add_from_text(u'', VERSE_FOR_LINE_COUNT)
         else:
             self.imageManager.del_image(theme_data.theme_name)
             serviceItem.add_from_text(u'', VERSE)