← Back to team overview

openlp-core team mailing list archive

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

Re: [Merge] lp:~sam92/openlp/bug-1332990 into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Needs Information

The code attempts to write to the theme file.
If it fails we have an open theme handle and an incomplete file.
Should not the fix be to close then delete the file if an exception is thrown?
-- 
https://code.launchpad.net/~sam92/openlp/bug-1332990/+merge/224132
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next