Review Request: Generate accessors (getters/setters) for C++

[Jonas Jacobi]

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

Review request for KDevelop.


Description
-------

Generate accessors (getters/setters) for C++.

This patch adds the possibility to create accessor methods for class member variables via the gui.
The dialog offers the following features:
- shows all members of a selected class in a tree view (with getters/setters as children), if it was opened on a class member that one is pre-selected
- only shows setters if applicable (non-const member)
- accessor method definitions can be created inline or separate (in the fitting source file, if applicable)
- configurable automatic creation of method names with manual override for each accessor method
- access policy (visibility) setting global and invidualy for each accessor method
- configurable default setter parameter name with manual override for each setter (void setFoo(int myParamterName))
- selectable getter return type (e.g. Type, Type&, const Type& for a member of type Type)
- create setters as slots for QObject subclasses

The code isn't polished and the patch contains whitespace issues in sourcemanipulation.cpp, but it's in a usable state and i would like to get some feedback on the dialog, especially on usability improvements.

Regards,
Jonas


Diffs
-----

  languages/cpp/CMakeLists.txt a1fb97c 
  languages/cpp/codegen/generateaccessorsdialog.h PRE-CREATION 
  languages/cpp/codegen/generateaccessorsdialog.cpp PRE-CREATION 
  languages/cpp/codegen/generateaccessorsmodel.h PRE-CREATION 
  languages/cpp/codegen/generateaccessorsmodel.cpp PRE-CREATION 
  languages/cpp/codegen/simplerefactoring.h b2187c3 
  languages/cpp/codegen/simplerefactoring.cpp fd62ccf 
  languages/cpp/codegen/ui/cppgenerateaccesors.ui PRE-CREATION 
  languages/cpp/cppduchain/missingdeclarationassistant.cpp bafc159 
  languages/cpp/cppduchain/sourcemanipulation.h 6f8a79b 
  languages/cpp/cppduchain/sourcemanipulation.cpp 5cb6a67 
  languages/cpp/tests/CMakeLists.txt 40d07b0 
  pics/mini/CMakeLists.txt c3316e7 
  pics/mini/private_slot.png PRE-CREATION 
  pics/mini/protected_slot.png PRE-CREATION 
  pics/mini/public_slot.png PRE-CREATION 

Diff: http://git.reviewboard.kde.org/r/103613/diff/diff


Testing
-------


Screenshots
-----------


  http://git.reviewboard.kde.org/r/103613/s/392/

  http://git.reviewboard.kde.org/r/103613/s/393/


Thanks,

Jonas Jacobi

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