[KParts] patch: don't emit urlChanged, when url is the same
Michal Humpula
michal.humpula at seznam.cz
Tue Dec 24 11:30:56 UTC 2013
Hi there,
as suggested without fully functional reviewboard, here is the little patch
for discussion.
Emit urlChanged only when actually changing url trough setUrl. The drawback of
this is that now the fast assignment is replaced by possibly expensive string
comparison. On the other hand, when the urls are the same, the emit wont fire
and whole bunch of potentially expensive operations connected to that signal
will not happen.
IMHO the same could/should be done for others setters.
Cheers
Michal
-------------- next part --------------
A non-text attachment was scrubbed...
Name: emit.patch
Type: text/x-patch
Size: 392 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20131224/c5ade969/attachment.patch>
More information about the Kde-frameworks-devel
mailing list