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