https://bugs.kde.org/show_bug.cgi?id=322922
--- Comment #12 from Mark <mark...@gmail.com> --- (In reply to Nate Graham from comment #10) > Thanks Mark. So if I'm understanding correctly, this is a possible > performance issue with exceptionally large directories? I agree that if this > is worth fixing, option 1 (optimizing KDirLister) is the right approach. Well, kind of. It's a bit of a grey area. It's weird to let KDirLister exclude .directory file changes. Some would consider that a bug while that very same "fix" would stop it to update the folder when the .directory file got changed in dolphin. Patching KDirLister for ".directory" handling certainly is a no-go! That file is for Dolphin and KDirLister should not know a thing about that. A "perhaps acceptable solution" on that side is something like a general "KDirLister::excludeFromMonitor(QString expression)" where the string is then fed into a QRegularExpression and anything matching the expression is excluded from being monitored. That "could" work... Feel free to implement that and hope it gets accepted ;) Another issue that will arise is different file system monitor backends. I know inotify supports what we want, but what about the range of others on linux alone? Then you still have those other platforms (Windows, Mac) they need to follow suit. The change for this is really not trivial it seems. In conclusion for the above.Having that in KDirLister would be a bonus! A big plus, but not a requirement. I agree with mappu04 that the .directory file should not be in an individual directory. Why would i need that file when i happen to copy a folder from my local drive to a completely different machine where i might not even run Dolphin. And if i were, i would certainly _not_ want that copy to look different on that other machine just because i accidentally copied over a .directory file! So, could you please set the title back to what it was and keep this about getting rid of the .directory file from a folder. It's fine if you don't agree, but then this issue should be a "won't fix". -- You are receiving this mail because: You are watching all bug changes.
