← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~crichter/openlp/shortcut-dialog into lp:openlp

 

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.



References