D5709: Implicitly shared class template: improve method declarations
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Thu May 4 13:48:58 UTC 2017
kossebau added a comment.
In https://phabricator.kde.org/D5709#106903, @mwolff wrote:
> http://doc.qt.io/qt-5/qshareddatapointer.html -> your change looks sane for *explicitly* shared data members, but not for implicit ones which will call detach in non-const `operator->()` automatically
>
> this also means that your setter will always detach, the equality check is really not needed and just code noise.
Eek, was not aware of that automatic detach for non-const methods. Will have to check some code of mine then :) Though first update of this patch...
REPOSITORY
R32 KDevelop
REVISION DETAIL
https://phabricator.kde.org/D5709
To: kossebau, #kdevelop, mwolff
Cc: mwolff, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170504/8149cb1f/attachment.html>
More information about the KDevelop-devel
mailing list