openlp-core team mailing list archive
-
openlp-core team
-
Mailing list archive
-
Message #05606
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