D18632: Port away from deprecated KIO::UDSEntry::insert()
Chinmoy Ranjan Pradhan
noreply at phabricator.kde.org
Tue Feb 5 16:32:09 GMT 2019
chinmoyr accepted this revision.
chinmoyr added a comment.
This revision is now accepted and ready to land.
LGTM
INLINE COMMENTS
> kossebau wrote in kio_desktop.cpp:183
> :)
>
> Well, I would rather be safe then sorry. We have no control over what other code might do to the entry object, also is there nothing noted in any API contracts.
> And it's not very performance critical code here, so I propose future-proof code for now.
I agree with you here.
REPOSITORY
R120 Plasma Workspace
BRANCH
portudsentryinsert
REVISION DETAIL
https://phabricator.kde.org/D18632
To: kossebau, #plasma, chinmoyr
Cc: chinmoyr, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190205/90e70ec3/attachment.html>
More information about the Plasma-devel
mailing list