Review: Needs Fixing
Remove rather than commenting out.

Sort out multitude of long lines.

Sort out parameter alignment.  We only indent 4 spaces for continuation like
function_call(parameter1, p2, p3)
or
function_call(
    longparameter1,
    longparameter2,
    etc)
not half way across the screen.
-- 
https://code.launchpad.net/~crichter/openlp/shortcut-dialog/+merge/46398
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to