dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


  "perhaps fix storedPut to do that?"  --> I'm not in favour. Generally 
speaking, we want to do this at the highest level possible, so that batching N 
jobs doesn't trigger N notifications.
  
  This is e.g. why CopyJob does it, but FileCopyJob doesn't.
  
  Granted we don't have jobs which do multiple storedPuts right now (IIRC), but 
I can imagine a day where this is the case.

REPOSITORY
  R241 KIO

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

To: broulik, #frameworks, dfaure
Cc: dfaure, ngraham, elvisangelaccio, kde-frameworks-devel, michaelh, bruns

Reply via email to