Review: Needs Fixing
Lines 142, 143, 144, "bible" should be "Bible".

Why did you remove line 152 and add it back in individually on lines 160 and 
166? That is unnecessary.

You need a one line description of the method, on line 312, even if it is just 
"Performs a text search on the current Bible."

Line 326 "separate" (it has an a, not an e).


-- 
https://code.launchpad.net/~googol-hush/openlp/bibles/+merge/38039
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to