On 09.10.2014 16:59, Michael Meeks wrote: > * Leaking /tmp files (Cloph) > + the 'lu' files are leaked by utl::TempFile (Miklos) > + tinderbox has a custom tmp (Norbert) > + creating a directory in workdir for tmp > + could we do it in the tinderbox script by default ? (Michael) > + there is an option to do that already (Norbert) > + would still be good not to leak (Miklos) > + we leak 1536 files per build (Norbert) > + volunteers appreciated; could it be document load unit tests not > cleaning up (?)
this is now significantly improved on master: on an incremental "make check" now only about 5 temp files / directories are leaked, none of which are from a CppunitTest; there are also various leaked ooopackaging* directories from the installer. and it was not just trivial missing close() in unit tests that caused the leakage; there were some real problems where temp files would leak document content for end users too, including one of the oldest regressions ever, introduced when converting tools DirEntry to osl::Directory on 2000-10-12 in commit f198ed89a _______________________________________________ List Name: Libreoffice-qa mailing list Mail address: Libreoffice-qa@lists.freedesktop.org Change settings: http://lists.freedesktop.org/mailman/listinfo/libreoffice-qa Problems? http://www.libreoffice.org/get-help/mailing-lists/how-to-unsubscribe/ Posting guidelines + more: http://wiki.documentfoundation.org/Netiquette List archive: http://lists.freedesktop.org/archives/libreoffice-qa/