Review Request 124369: Encapsulate field refactoring

Maciej Poleski d82ks8djf82msd83hf8sc02lqb5gh5 at gmail.com
Mon Jul 20 22:37:17 UTC 2015


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

(Updated Lip 21, 2015, 12:37 rano)


Review request for KDevelop.


Changes
-------

Finished implementation (small improvements required)


Summary (updated)
-----------------

Encapsulate field refactoring


Repository: kdev-clang


Description (updated)
-------

Encapsulate field refactoring

- Displays dialog
- Transforms "use" site - replaces direct uses of encapsulated variable by calls to getter and setter method/function.
- Generates getter and setter implementation

Attached image presents GUI with its default content (prefilled by heuristics on DeclaratorDecl AST node).


Diffs (updated)
-----

  refactoring/CMakeLists.txt PRE-CREATION 
  refactoring/encapsulatefielddialog.h PRE-CREATION 
  refactoring/encapsulatefielddialog.cpp PRE-CREATION 
  refactoring/encapsulatefielddialog.ui PRE-CREATION 
  refactoring/encapsulatefieldrefactoring.h PRE-CREATION 
  refactoring/encapsulatefieldrefactoring.cpp PRE-CREATION 
  refactoring/encapsulatefieldrefactoring_changepack.h PRE-CREATION 
  refactoring/encapsulatefieldrefactoring_changepack.cpp PRE-CREATION 
  refactoring/redeclarationchain.cpp PRE-CREATION 
  refactoring/refactoringmanager.cpp PRE-CREATION 
  refactoring/tudecldispatcher.h PRE-CREATION 
  refactoring/tudecldispatcher.cpp PRE-CREATION 
  refactoring/utils.h PRE-CREATION 
  refactoring/utils.cpp PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/124369/diff/


Testing (updated)
-------

Manual testing, tests are coming (tomorrow?)


File Attachments
----------------

Encapsulate field dialog
  https://git.reviewboard.kde.org/media/uploaded/files/2015/07/15/ea15c55b-cc4d-4dfa-ac34-8c568a93701d__snapshot3.png


Thanks,

Maciej Poleski

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


More information about the KDevelop-devel mailing list