D18698: [baloo/KInotify] Notify if folder was moved from unwatched place
    Stefan BrĂ¼ns 
    noreply at phabricator.kde.org
       
    Fri Feb 15 03:11:23 GMT 2019
    
    
  
bruns added inline comments.
INLINE COMMENTS
> poboiko wrote in kinotify.cpp:390
> Right, sorry, misprinted. 
> Actually, since there are a lot of `decodeName` calls around, probably it would be better to decode it just once, right before `event->mask` matching then, and then pass it everywhere?
> (less code duplication & mutliple calls if `mask` matches several events...)
Yes, but such a cleanup should go into a different review.
REPOSITORY
  R293 Baloo
REVISION DETAIL
  https://phabricator.kde.org/D18698
To: poboiko, #baloo, #frameworks, ngraham, bruns
Cc: bruns, ngraham, kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190215/df0a5e79/attachment.html>
    
    
More information about the Kde-frameworks-devel
mailing list