aacid created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. aacid requested review of this revision.
REVISION SUMMARY I was wondering if adding this check in KArchive::setRootDir instead but i ended doing it here since i think an assert in setRootDir is valid for when C++ users of that class do it wrong. REPOSITORY R243 KArchive BRANCH master REVISION DETAIL https://phabricator.kde.org/D22786 AFFECTED FILES src/karchive_p.h src/ktar.cpp To: aacid Cc: kde-frameworks-devel, LeGast00n, sbergeron, michaelh, ngraham, bruns