Review Request: Remove unnecessary code in abstractusebuilder.h

Sven Brauch svenbrauch at gmx.de
Tue Mar 15 00:21:17 UTC 2011


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

Review request for KDevelop.


Summary
-------

Just stumbled about this apparently pointless bool (it's created locally, but never changed and once checked with if ( ! encountered ), which is always true). Trying to understand the code it is involved in, I find it confusing, so I'd propose to remove it.


Diffs
-----

  language/duchain/builders/abstractusebuilder.h ff8f44e 

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


Testing
-------


Thanks,

Sven

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


More information about the KDevelop-devel mailing list