[Differential] [Request, 63 lines] D2265: Fix preview of nested archives

[elvisangelaccio (Elvis Angelaccio)]

elvisangelaccio created this revision.
elvisangelaccio added a reviewer: rthomsen.
Restricted Application added a subscriber: kde-utils-devel.
Restricted Application added a project: Ark.

REVISION SUMMARY
  My assumption that a PreviewJob can remove its temp dir when it's done was
  wrong, at least with nested archives. If we remove the extracted file too
  soon, there is no way to preview a nested archive or, worse, a "double-nested"
  archive.
  
  This patch just delays the cleanup to the Part destructor (= when we are
  closing the app).

TEST PLAN
  - Open https://go.skype.com/skypeforlinux-64-alpha.deb
  - Preview of data.tar.xz now works.
  - Make sure that the extracted file in /tmp is deleted on Ark quit.

REPOSITORY
  rARK Ark

BRANCH
  fix-nested-preview

REVISION DETAIL
  https://phabricator.kde.org/D2265

AFFECTED FILES
  kerfuffle/jobs.cpp
  kerfuffle/jobs.h
  part/part.cpp
  part/part.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: elvisangelaccio, rthomsen
Cc: kde-utils-devel, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160722/0ed1044d/attachment.html>