Review Request: New testcase for ExpressionParser

Bertjan Broeksema b.broeksema at home.nl
Thu Oct 1 09:37:13 UTC 2009


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

(Updated 2009-10-01 09:37:13.175997)


Review request for KDevelop and David Nolden.


Changes
-------

Felt like fixing it. So I present a fix for the bug (also fixes volatile modifier). Additionally I merged the code with the TypeASTVisitor as mentioned in the code to prevent further duplication.


Summary
-------

A test case to show incorrect parsing of "const char*" to "char *" in ExpressionParser.


Diffs (updated)
-----

  trunk/KDE/kdevelop/languages/cpp/cppduchain/expressionvisitor.cpp 1029180 
  trunk/KDE/kdevelop/languages/cpp/cppduchain/tests/test_expressionparser.h 1029661 
  trunk/KDE/kdevelop/languages/cpp/cppduchain/tests/test_expressionparser.cpp 1029661 
  trunk/KDE/kdevelop/languages/cpp/cppduchain/type_visitor.cpp 1029180 

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


Testing (updated)
-------

Run all cppduchain tests. Those work fine except for the cppcodegentest, but as mentioned before, these seems b0rked anyhow.


Thanks,

Bertjan





More information about the KDevelop-devel mailing list