D25565: Optimize ArchiveModel::cleanFileName

[Elvis Angelaccio]

elvisangelaccio added a comment.


  Interesting. How did you test this? Just opening an archive?

INLINE COMMENTS

> archivemodel.cpp:346
> -    if (fileName.contains(pattern, &match) && match.captured() == fileName) {
> -        qCDebug(ARK) << "Skipping entry with filename" << fileName;
>          return QString();

Why remove the debug line though?

REPOSITORY
  R36 Ark

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

To: nicolasfella, #ark
Cc: elvisangelaccio, kde-utils-devel, fbampaloukas, tctara
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20191128/874dc0b0/attachment.html>