I was following line 38 !
--
https://code.launchpad.net/~trb143/openlp/remotes/+merge/102150
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Unsu
Review: Approve
--
https://code.launchpad.net/~raoul-snyman/openlp/web-remote/+merge/102194
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: https://launchpad.net/~openlp-core
Post to : openlp-core@lists.launchpad.net
Uns
Review: Needs Information
What is the "or 0" for on line 37? Since it's initialised to 0 on line 8, is it
necessary?
--
https://code.launchpad.net/~trb143/openlp/remotes/+merge/102150
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailin
Changes are fine, but just wondering if there should be a "if not remote:"
prior to line 138 as per 125?
--
https://code.launchpad.net/~googol/openlp/trivial2/+merge/102059
Your team OpenLP Core is subscribed to branch lp:openlp.
___
Mailing list: htt
Review: Approve
--
https://code.launchpad.net/~googol/openlp/trivial/+merge/102027
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
Tim Bentley has proposed merging lp:~trb143/openlp/remotes into lp:openlp.
Requested reviews:
OpenLP Core (openlp-core)
For more details, see:
https://code.launchpad.net/~trb143/openlp/remotes/+merge/102150
Add tag to allow the Android client to see if a service list has changed since
it's la
Review: Approve
--
https://code.launchpad.net/~googol/openlp/trivial2/+merge/102059
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
The proposal to merge lp:~googol/openlp/trivial2 into lp:openlp has been
updated.
Description changed to:
Hello,
- trivial clean ups
Note: In the lines 62-143 I moved the for loop out of the if.
For more details, see:
https://code.launchpad.net/~googol/openlp/trivial2/+merge/102059
--
https:
8 matches
Mail list logo