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


  Too bad we have so many lines to change.
  
  Could you maybe add some context to your commit comment like `activities: 
Close the database connection to prevent file descriptor leak`
  
  LGTM

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

To: akandaurov, ivan, meven
Cc: meven, kde-frameworks-devel, kfm-devel, azyx, nikolaik, pberestov, 
iasensio, fprice, LeGast00n, cblack, fbampaloukas, alexde, GB_2, Codezela, 
feverfew, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, 
bruns, emmanuelp, rdieter, mikesomov

Reply via email to