D7700: Show list of tags in PlacesView

Nicolas Fella noreply at phabricator.kde.org
Tue Nov 13 17:05:30 GMT 2018


nicolasfella added inline comments.

INLINE COMMENTS

> ngraham wrote in kfileplacesmodel.cpp:187
> const. But in fact, does this even need to be a variable? It's only used once.

Will fix on push

REPOSITORY
  R241 KIO

BRANCH
  arcpatch-D7700

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

To: nicolasfella, #dolphin, #kde_applications, alexeymin, ngraham
Cc: kde-frameworks-devel, bruns, rkflx, mmustac, spoorun, michaelh, renatoo, anthonyfieroni, cfeck, elvisangelaccio, emmanuelp, ngraham, alexeymin, #dolphin, alexde, sourabhboss, feverfew, navarromorales, firef, andrebarros, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20181113/fb5a840c/attachment.htm>


More information about the kfm-devel mailing list