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