openlp-core team mailing list archive

[Raoul Snyman <raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx>]

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.