D28739: Close the database connection to prevent file descriptor leak

Alexander Kandaurov noreply at phabricator.kde.org
Sat Apr 11 13:14:41 BST 2020


akandaurov updated this revision to Diff 79827.
akandaurov added a comment.


  Added indentation. The block is suggested by the Qt Documentation <https://doc.qt.io/qt-5/qsqldatabase.html#removeDatabase>. Without it, a warning will be thrown:
  
    QSqlDatabasePrivate::removeDatabase: connection 'kactivities_db_resources_94156823977952' is still in use, all queries will cease to work.

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D28739?vs=79808&id=79827

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

AFFECTED FILES
  activities/fileitemplugin/FileItemLinkingPluginActionLoader.cpp

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/ecce48d4/attachment.htm>


More information about the kfm-devel mailing list