← Back to team overview

openlp-core team mailing list archive

  1. openlp-core team
  2. Mailing list archive
  3. Message #12465

Re: [Merge] lp:~smpettit/openlp/inno-uninstall into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Approve

It is not cleaner. Code should never be repeated unless it is more than 200% necessary.

However, since you say there is no way around this, I am satisfied with it.
-- 
https://code.launchpad.net/~smpettit/openlp/inno-uninstall/+merge/79121
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next