Bug 63133 - write tools/tempfiles.hxx out
Summary: write tools/tempfiles.hxx out
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Libreoffice (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: palenik.mihaly
QA Contact:
URL:
Whiteboard: EasyHack,DifficultyInteresting,SkillC...
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-04 19:32 UTC by Andras Timar
Modified: 2013-07-16 18:46 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Andras Timar 2013-04-04 19:32:06 UTC
Background: tools is a library packed full of deprecated code, that has equivalents in sal/ (the system abstraction layer). Of course the sal/ versions are often less usable than the tools/ version, so some API improvements may be required on the way too.

tools/tempfiles.hxx defines the TempFile class, which is still used at some random places in the codebase. To find them all:

git grep "<tools/tempfile.hxx>"

The equivalent class from sal is osl::FileBase::createTempFile.
Comment 1 Andras Timar 2013-04-04 20:15:34 UTC
Having a second look at the code, I don't think it is a great idea to kill this. Not a big win at least.
Comment 2 Michael Stahl 2013-04-04 20:54:15 UTC
actually it should be very easy to kill tools TempFile because
it has a clone in unotools called utl::TempFile...

...which is not that much better of course but it's an easy win.

perhaps something better would make sense in the URE...
but currently the createTempFile there is incomplete
and difficult to use since it's apparently not even possible
to create an osl::File from the oslFileHandle it returns.
Comment 3 Commit Notification 2013-07-16 18:46:18 UTC
Palenik Mihaly committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=9c0a390489dd99aee4beaf73845740776b8083aa

fdo#63133: Write tools/tempfile.hxx out



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.