Code review comment for lp:~phill-ridout/openlp/bug1098075_2.0

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

Looks alright to me. I probably would have named the return value "result", but that's neither here nor there.

review: Approve

« Back to merge proposal