Review Request 112433: Use operator bool() instead of != and == for shared_ptr.
Commit Hook
null at kde.org
Tue Sep 3 11:00:49 BST 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112433/
-----------------------------------------------------------
(Updated Sept. 3, 2013, 10 a.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Multimedia and Christian Esken.
Description
-------
(This is part 1 of 2 of the shared_ptr changes)
In preparation for supporting C++11's version of shared_ptr, convert
some
comparisons to operator bool(), that is
if (foo != 0) becomes if (foo)
if (foo == 0) becomes if (!foo)
as otherwise the build (with clang and libc++) would fail because there
is
no overload for operator==(shared_ptr, int) and operator!=(shared_ptr,
int).
Diffs
-----
apps/kmixd.cpp 442abaf89b65f96382bf954e6cb592d61ff12b12
backends/mixer_backend.cpp 2e2e901552ec6213e66ffeb735e896f32563cb04
backends/mixer_mpris2.cpp 6ebcd8fcdb038d51d196356230070ccc634230e0
core/mixertoolbox.cpp 60c9fc8788cbe0c73efcad0e3291bafa20f8bdc7
gui/kmixdockwidget.cpp 47e807337f46e67750174c6fc5f938fc889a089c
gui/viewdockareapopup.cpp 48411bd161601b5301ba471a1f8103e4c40017dd
Diff: http://git.reviewboard.kde.org/r/112433/diff/
Testing
-------
Together with <https://git.reviewboard.kde.org/r/112434> building KMix with clang and libc++ finally succeeds.
Thanks,
Raphael Kubo da Costa
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20130903/54ea3a6a/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-multimedia mailing list
kde-multimedia at kde.org
https://mail.kde.org/mailman/listinfo/kde-multimedia
More information about the kde-multimedia
mailing list