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