← Back to team overview

openlp-core team mailing list archive

Thread
Date

Re: [Merge] lp:~googol/openlp/renderer-empty-theme into lp:openlp

To: mp+173326@xxxxxxxxxxxxxxxxxx
From: Andreas Preikschat <andreaspreikschat@xxxxxxxxxx>
Date: Sat, 06 Jul 2013 20:28:24 -0000
In-reply-to: <20130706193547.15818.3367.codereview@chaenomeles.canonical.com>
Reply-to: mp+173326@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

> Should you not be fixing the case not the effect?

I had the same thought, but then I though that this is more secure. (Rather have a robust core api).

Agreed?
-- 
https://code.launchpad.net/~googol/openlp/renderer-empty-theme/+merge/173326
Your team OpenLP Core is requested to review the proposed merge of lp:~googol/openlp/renderer-empty-theme into lp:openlp.

References

Re: [Merge] lp:~googol/openlp/renderer-empty-theme into lp:openlp
From: Tim Bentley, 2013-07-06