Review Request 124624: Extract function refactoring
Aleix Pol Gonzalez
aleixpol at kde.org
Wed Aug 5 13:47:23 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124624/#review83460
-----------------------------------------------------------
Looks simple enough, can we get a test? :D
- Aleix Pol Gonzalez
On Aug. 5, 2015, 2:06 a.m., Maciej Poleski wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124624/
> -----------------------------------------------------------
>
> (Updated Aug. 5, 2015, 2:06 a.m.)
>
>
> Review request for KDevelop.
>
>
> Repository: kdev-clang
>
>
> Description
> -------
>
> Extract function (from expression) refactoring
>
> Unfinished (one FIXME and one TODO, but these are small changes).
>
> This refactoring extracts function from expression. More advanced version would also allow extracting from statements (but have to handle input _and_ __output__ variables).
>
>
> Diffs
> -----
>
> refactoring/CMakeLists.txt PRE-CREATION
> refactoring/extractfunctionrefactoring.h PRE-CREATION
> refactoring/extractfunctionrefactoring.cpp PRE-CREATION
> refactoring/refactoringmanager.cpp PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/124624/diff/
>
>
> Testing
> -------
>
> Manual testing (produces invalid code because of FIXME)
>
>
> Thanks,
>
> Maciej Poleski
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150805/81cec348/attachment.html>
More information about the KDevelop-devel
mailing list