D28739: Close the database connection to prevent file descriptor leak

Méven Car noreply at phabricator.kde.org
Sat Apr 11 14:25:38 BST 2020


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200411/72b7cc8b/attachment.htm>


More information about the kfm-devel mailing list