Review Request 124256: Introduce rename field refactoring
Maciej Poleski
d82ks8djf82msd83hf8sc02lqb5gh5 at gmail.com
Sat Jul 11 22:46:20 UTC 2015
> On Lip 11, 2015, 5:35 po południu, Milian Wolff wrote:
> > Can you *please* start working on a unit test framework? I'm very reluctant of integrating patches for stuff that is not properly tested. Esp. in your area, having tests is crucial to document the implemented behavior, and also verify it keeps functional in the future.
https://git.reviewboard.kde.org/r/124329/
- Maciej
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124256/#review82355
-----------------------------------------------------------
On Lip 10, 2015, 8:12 po południu, Maciej Poleski wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124256/
> -----------------------------------------------------------
>
> (Updated Lip 10, 2015, 8:12 po południu)
>
>
> Review request for KDevelop.
>
>
> Repository: kdev-clang
>
>
> Description
> -------
>
> Renaming of fields with external linkage.
>
> Renaming of fields from classes in anonymous namespaces is also done
>
>
> Diffs
> -----
>
> refactoring/CMakeLists.txt PRE-CREATION
> refactoring/refactoringmanager.cpp PRE-CREATION
> refactoring/renamefielddeclrefactoring.h PRE-CREATION
> refactoring/renamefielddeclrefactoring.cpp PRE-CREATION
> refactoring/renamefielddeclturefactoring.h PRE-CREATION
> refactoring/renamefielddeclturefactoring.cpp PRE-CREATION
> refactoring/renamevardeclrefactoring.h PRE-CREATION
> refactoring/renamevardeclrefactoring.cpp PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/124256/diff/
>
>
> Testing
> -------
>
> Manual testing
>
>
> Thanks,
>
> Maciej Poleski
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150711/5d394b70/attachment.html>
More information about the KDevelop-devel
mailing list