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


  Seems hard to miss cases like for `// +1 for UDS_DISPLAY_NAME` but I will 
trust you did enough review to be confident for all those cases.
  A missing one will just cause an allocation anyway which is what this is 
fixing in the first place, so no arm done basically.

REPOSITORY
  R320 KIO Extras

BRANCH
  usemoreudsentryreserve

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

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

Reply via email to