openlp-core team mailing list archive

[Martin Thompson <martin@xxxxxxxxxxxxxxxxxx>]

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
-- 
martin@xxxxxxxxxxxxxxxxxx
  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.