KTempFile->QTemporaryFile

[Thiago Macieira]

Jaison Lee wrote:
>For all these reasons, I think we should create a new very small class
>called KTemporaryFile that inherits from QTemporaryFile. It will be in
>charge of setting the desired tmp directory, as well as setting
>autoremove false (if we decide we want to keep that behaviour). It
>will also provide a good place to put equivalents of the textStream()
>and dataStream() functions which I think will be handy shortcuts and
>make porting a bit easier.
>
>My plan then would be:
>
>Step 1) Create KTemporaryFile and mark KTempFile deprecated.
>Step 2) Port all existing KTempFiles to KTemporaryFile.
>Step 3) Remove KTempFile.
>
>This will provide Will and I ample time to complete step 2, while
>making sure that the codebase remains able to be compiled.
>
>So how does this sound to everyone? Does this look like something
>worth doing? Any comments on my overall plan? Should KTemporaryFile
>still default autoRemove to false even though QTemporaryFile defaults
>true? (Personally I would say no; take the Qt default in this case.)

I don't know about the auto-remove default. It looks to me like it should 
auto-remove: my $KDETMP is filled with stale temporary files.

In any event, the directory makes it already a necessity to have a 
specialisation of KTemporaryFile, so I agree with your proposal.

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20060910/b83f6244/attachment.sig>