karchive plugin extract all fix

Review Request #102960 - Created Oct. 24, 2011 and submitted

Information
Theofilos Intzoglou
ark
Reviewers
kdeutils
rkcosta
* If no item was selected from the archive, nothing was extracted! Now it extracts everything.
* Check if an entry is a directory and append '/' if necessary

  

Issues

  • 4
  • 12
  • 0
  • 16
Description From Last Updated
Do you mean adding a QDir(filepath).exists() check or keeping created paths in memory? Theofilos Intzoglou Theofilos Intzoglou
Normally, it shouldn't be possible to get to that return value. Sometimes I get a bit paranoid with the checks ... Theofilos Intzoglou Theofilos Intzoglou
QString() is sufficient (and probably cheaper). Raphael Kubo da Costa Raphael Kubo da Costa
Ditto. Raphael Kubo da Costa Raphael Kubo da Costa
Raphael Kubo da Costa
Theofilos Intzoglou
Raphael Kubo da Costa
Theofilos Intzoglou
Theofilos Intzoglou
Raphael Kubo da Costa
Theofilos Intzoglou
Raphael Kubo da Costa
Commit Hook
Theofilos Intzoglou
Review request changed

Status: Closed (submitted)

Loading...