Review Request 124450: Restore std::move tests, revert part of cf48e4d26257b

Boris Egorov egorov at linux.com
Fri Jul 24 16:46:32 UTC 2015



> On July 24, 2015, 4:36 p.m., Milian Wolff wrote:
> > yes, please push this ASAP (does it also fix the kdevelop issues?). then I'll try to fix the issues the coming days.
> > 
> > Thanks!

No, it only solves kdevplatform issues. Fixes for kdevelop will follow.


- Boris


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124450/#review82906
-----------------------------------------------------------


On July 24, 2015, 11:59 a.m., Boris Egorov wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124450/
> -----------------------------------------------------------
> 
> (Updated July 24, 2015, 11:59 a.m.)
> 
> 
> Review request for KDevelop, Kevin Funk and Milian Wolff.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> Commit cf48e4d26257b removed tests of 'moved-from' values after
> std::move. This commit restores these tests and make move constructors
> non-explicit to pass tests. Behaviour of explicit move constructors
> should be investigated later.
> 
> See https://git.reviewboard.kde.org/r/124427/
> 
> Note: there is one more move constructor which is still explicit:
> 
> explicit OutlineNode(OutlineNode&& other) noexcept;
> 
> There is no test for it, so change making them explicit was not reverted.
> We should add a test for this constructor in the first order, IMHO.
> 
> 
> Diffs
> -----
> 
>   language/duchain/identifier.h 204f8e06ba4f73da49df3a971a90b84745a1712f 
>   language/duchain/tests/test_identifier.cpp 0506fdbc999e5a14aaf8599376f4f84cb94affc4 
>   serialization/indexedstring.h 4c8ed65d8c6fd0f8fa7bc1edab437d5ba15837d9 
>   serialization/tests/test_indexedstring.cpp 19ce452567eefc4bc08ad3087a8dc15919a75fc9 
> 
> Diff: https://git.reviewboard.kde.org/r/124450/diff/
> 
> 
> Testing
> -------
> 
> Builds, passes tests
> 
> 
> File Attachments
> ----------------
> 
> failed tests verbose log
>   https://git.reviewboard.kde.org/media/uploaded/files/2015/07/24/f8fc5841-8203-421c-ac11-513d45fd97fc__kdevplatform_explicit_failed_tests.log
> 
> 
> Thanks,
> 
> Boris Egorov
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150724/0d625c16/attachment.html>


More information about the KDevelop-devel mailing list