Review Request 111531: Fix CodeAnalysisTest

Milian Wolff mail at milianw.de
Tue Jul 16 12:18:19 UTC 2013


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

Review request for KDevelop and Aleix Pol Gonzalez.


Description
-------

The unit test is broken, I think b/c the logic is wrong in the code, i.e. it checked for const and applied read/write to it, whereas non-const got only read access.

Also, how should the stuff with the "do I have an argument or not?" work? the test used to have an operator++(int) or so which simply did nothing?


Diffs
-----

  languages/cpp/cppduchain/tests/codeanalysistest.cpp e77f961 
  languages/cpp/cppduchain/usedecoratorvisitor.cpp 57dfd88 

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


Testing
-------

unit tests pass, but are not checking the operator++(int) case anymore.


Thanks,

Milian Wolff

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


More information about the KDevelop-devel mailing list