Review Request 129341: [kwallet] boost::shared_ptr -> std::shared_ptr
David Faure
faure at kde.org
Sun Dec 4 09:17:34 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129341/#review101264
-----------------------------------------------------------
Ship it!
Looks OK. It would have been better to make this two separate commits though (one about shared_ptr and one about the #define, they are completely unrelated).
- David Faure
On Nov. 15, 2016, 8:54 p.m., Andreas Sturmlechner wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129341/
> -----------------------------------------------------------
>
> (Updated Nov. 15, 2016, 8:54 p.m.)
>
>
> Review request for KDE Frameworks.
>
>
> Repository: kwallet
>
>
> Description
> -------
>
> Dropping implicit boost usage.
> Also, rename HAVE_QGPGME to HAVE_GPGMEPP.
>
>
> Diffs
> -----
>
> src/runtime/kwalletd/CMakeLists.txt 5092a7e77a1ca7203827e444c11310566b2cc8b5
> src/runtime/kwalletd/backend/CMakeLists.txt 2f99bc3cca89dd7a194f46cb314d2f7f90f931f3
> src/runtime/kwalletd/backend/backendpersisthandler.h a6682353d62fd63e88a0de6277801c624c794c74
> src/runtime/kwalletd/backend/backendpersisthandler.cpp 413d4cd23ef1e15b75cd366061f07f179241af58
> src/runtime/kwalletd/backend/kwalletbackend.h 4811a58e942685b01ed097ef15c2838e3833865c
> src/runtime/kwalletd/backend/kwalletbackend.cc 43f30a545d36b64b102c794de3625621a694983e
> src/runtime/kwalletd/knewwalletdialog.h ea71fbc7d59fdd05da7ce395e371dd7fdf91e49d
> src/runtime/kwalletd/knewwalletdialog.cpp 0d9ffb8ed469536cee9d4055ac0bd001fff115a0
> src/runtime/kwalletd/kwalletd.cpp ca7d4ffbdff4178878b3c705921ef2b1970e0dbd
> src/runtime/kwalletd/kwalletwizard.h 4cf8de424af800716e7c779d6d419eae4b54e426
> src/runtime/kwalletd/kwalletwizard.cpp cd52e54a668554fc0e14f4ad4cb29ba6f27fd330
>
> Diff: https://git.reviewboard.kde.org/r/129341/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Andreas Sturmlechner
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20161204/12991fcb/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list