elvisangelaccio added inline comments. INLINE COMMENTS
> kfilewidget.cpp:132 > void updateLocationEditExtension(const QString &); > + bool matchFilter(const QString &filename, const QString &filter, const > bool bUpdate); > void updateFilter(); Nitpick: I'd make `filter` the first parameter. And we usually don't use `const boll` in signatures, but just `bool`. `bUpdate` doesn't tell the reader what this variable is used for. How about calling it `updateCurrentFilter` or similar? REPOSITORY R241 KIO REVISION DETAIL https://phabricator.kde.org/D21249 To: jglogowski, #frameworks, ngraham Cc: elvisangelaccio, ngraham, michaelweghorn, kde-frameworks-devel, michaelh, bruns