openlp-core team mailing list archive

Thread
Date

Re: [Merge] lp:~gerald-britton/openlp/newbugs into lp:openlp

To: jerryb <gerald.britton@xxxxxxxxx>
From: Andreas Preikschat <andreaspreikschat@xxxxxxxxxx>
Date: Thu, 26 May 2011 15:45:31 -0000
In-reply-to: <20110526134602.16926.60532.launchpad@loganberry.canonical.com>
Reply-to: mp+62484@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing
Hello,

Please remove the lambda. It is rather confusing than helpful (the call has to be made anyway). And  'Unable to open file' is not the right error message when the file does not exist. (Btw, we don't use unicode strings for strings in translate).

Sorry...
-- 
https://code.launchpad.net/~gerald-britton/openlp/newbugs/+merge/62484
Your team OpenLP Core is subscribed to branch lp:openlp.

Follow ups

Re: [Merge] lp:~gerald-britton/openlp/newbugs into lp:openlp
From: jerryb, 2011-05-26

References

[Merge] lp:~gerald-britton/openlp/newbugs into lp:openlp
From: jerryb, 2011-05-26