On Wed, Jan 27, 2010 at 12:40:31PM -0000 or thereabouts, Raoul Snyman wrote:
> Raoul Snyman has proposed merging lp:~raoul-snyman/openlp/biblescript into 
> lp:openlp.
> 
>     Requested reviews:
>     OpenLP Core (openlp-core)
>

Just one quick comment - why not use the python logging framework instead of 
> +    if debug or verbose:
> +        print 'Creating new database:'

?

Cheers,
Martin
-- 
[email protected]
  A man leaves his darkness when he follows the Son
      - The Great American Novel (Larry Norman)
https://code.launchpad.net/~raoul-snyman/openlp/biblescript/+merge/18132
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to