Fix confused by the different with QTemporaryDir and QDir issue

Review Request #130212 - Created Aug. 2, 2017 and discarded

Information
Leslie Zhai
k3b
master
383011
Reviewers
kdemultimedia
aacid, anthonyfieroni, sitter

Hi KDE developers,

tempDir was defined as QScopedPointer<QTemporaryDir> https://github.com/KDE/k3b/commit/05cd636c9dfb35fdeea666b581c1f2b9c3820a90#diff-920d85bf84619674590c2e13a0428d00R40 it is easy to create a unique directory for temporary use, and QScopedPointer's destructor will free the allocated memory automatically, brain-free :)

but it still mkdir all parenets directories in complicated way https://github.com/KDE/k3b/commit/05cd636c9dfb35fdeea666b581c1f2b9c3820a90#diff-920d85bf84619674590c2e13a0428d00R113

so I just mkpath for temporary VIDEO_TS directory and provide correct temporary path, such as /tmp/k3bVideoDvdfINBwV to genisoimage, please review my patch, and give me some suggestion, thanks a lot!

Regards,
Leslie Zhai


  
Harald Sitter
Leslie Zhai
Review request changed

Status: Discarded

Loading...