Review Request: Do not bail out on rename if src==dest
Kevin Funk
krf at gmx.de
Tue Dec 4 23:57:43 UTC 2012
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107585/
-----------------------------------------------------------
Review request for KDevelop and Andreas Pakulat.
Description
-------
Do not bail out on rename if src==dest
* In case source and destination are equal, just return RenameOk
(this is the same behavior as std::rename has, when src==dest)
* Simplify both folder item's and file item's rename method by
returning early in case the rename target is invalid
* Add new tests for folder, build folder and file items
BUG: 311151
Diffs
-----
project/projectmodel.cpp 613ddb512d0f1909d5e74451146bf3138421a6a0
project/tests/projectmodeltest.cpp 4277b0c7e5b607fc9b50b9e0b24370ff0bdcd22f
Diff: http://git.reviewboard.kde.org/r/107585/diff/
Testing
-------
Thanks,
Kevin Funk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20121204/8fef618c/attachment.html>
More information about the KDevelop-devel
mailing list