Basic C++0x patch

David Nolden zwabel at googlemail.com
Sun Mar 14 08:26:52 UTC 2010


Hi, thanks a lot!

I'm not yet familiar with c++ 0x, so it would be great if you could also write 
some small tests to show that the stuff is working, to show what _exactly_ 
this patch is supposed to fix, and to make sure that it will stay working 
after future patches.

Take a look at kdevelop/languages/cpp/cppduchain/tests/test_duchain.cpp to see 
how the duchain tests work. If you just want to test your changes on the 
_parser_ level, (eg. make sure that the AST is built up correctly), you can 
add a test to kdevelop/languages/cpp/parser/tests/parsertest instead.

Also, there is a large set of already existing unit tests for the C++ support, 
so after any change to it, you should make sure that the existing tests are 
still succeeding. For that, you have to run 
kdevelop/languages/cpp/cppduchain/tests/duchaintest, 
kdevelop/languages/cpp/cppduchain/tests/cppexpressionparsertest, 
kdevelop/languages/cpp/tests/cppcodecompletiontest, 
kdevelop/languages/cpp/tests/cppcodegentest. And when you changed something 
about the parser, you should also run 
kdevelop/languages/cpp/parser/tests/parsertest.

Greetings, David




More information about the KDevelop-devel mailing list