Review Request: Add support for parsing Q_PROPERTY declarations

Tobias Koenig tokoe at kde.org
Thu May 6 08:58:57 UTC 2010


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

(Updated 2010-05-06 08:58:57.113605)


Review request for KDevelop and David Nolden.


Changes
-------

Updated the patch according to milian's comments


Summary
-------

Add support for parsing Q_PROPERTY declarations to the c++ parser

Please review for later inclusion into 4.0.0 release.


Diffs (updated)
-----

  /trunk/extragear/sdk/kdevelop/languages/cpp/cppduchain/CMakeLists.txt 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/cppduchain/declarationbuilder.h 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/cppduchain/declarationbuilder.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/cppduchain/qpropertydeclaration.h PRE-CREATION 
  /trunk/extragear/sdk/kdevelop/languages/cpp/cppduchain/qpropertydeclaration.cpp PRE-CREATION 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/ast.h 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/default_visitor.h 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/default_visitor.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/dumptree.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/lexer.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/parser.h 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/parser.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/tests/test_parser.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/tokens.h 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/tokens.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/visitor.h 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/parser/visitor.cpp 1122579 
  /trunk/extragear/sdk/kdevelop/languages/cpp/setuphelpers.cpp 1122579 

Diff: http://reviewboard.kde.org/r/3814/diff


Testing
-------

run unit tests


Thanks,

Tobias





More information about the KDevelop-devel mailing list