replace mkdtemp for more portabilitiy

Review Request #103692 - Created Jan. 13, 2012 and updated

Information
Patrick Spendrin
gwenview
KDE/4.8
Reviewers
gwenview
dfaure, lukas

since mkdtemp is not portable, I want to replace the code in question with the usage of KTempDir.

it builds well. Since I have no idea how that code is used, I hope you can give me some hints?

Issues

  • 1
  • 0
  • 0
  • 1
Description From Last Updated
The directory gets deleted when the KTempDir goes out of scope. The best fix would be to replace the *calls* ... David Faure David Faure
David Faure
Patrick Spendrin
Aurélien Gâteau
David Faure
Albert Astals Cid
Patrick Spendrin
Review request changed

Change Summary:

Lukas (new gwenview maintainer) - can you check this? The patch looks good to me, but I don't know how to test it. Of course in the frameworks branch, one can use my QTemporaryDir instead :-)

People:

+lukas
David Faure

(that last message was from me - don't trust the From field... I guess I can add people as CC because I'm admin of some sorts... gerrit allows everyone to add reviewers.... just saying....)

Lukáš Tinkl

I'd prefer the code be ported to QTemporaryDir instead, now that we're in the frameworks branch :)

Loading...