search

berkus / include-what-you-use

Automatically exported from code.google.com/p/include-what-you-use
Other
1 stars 0 forks source link

false positives due to Camel Case includes #152

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
iwyu is suggesting me to remove <QUrl> and add "qurl.h"

$ cat /usr/include/qt/QtCore/QUrl
#include "qurl.h"

they are synonyms.

Original issue reported on code.google.com by iamser...@gmail.com on 18 Jun 2014 at 9:14

GoogleCodeExporter commented 9 years ago 
Thank you, iamsergio for the report.  Does Qt document which style is 
preferred, <QUrl> or "qurl.h"?

I understand that you might prefer one style over another, but, strictly 
speaking, IWYU is correct.  You aren't using anything from QUrl, you are using 
something from qurl.h, and if you want to include what you use, you have to 
include qurl.h.

I doubt that IWYU will be able to detect "synonym" files, the best approach 
here is IWYU mappings.  See 
https://code.google.com/p/include-what-you-use/wiki/IWYUMappings for more 
details.

Original comment by vsap...@gmail.com on 19 Jun 2014 at 10:17

GoogleCodeExporter commented 9 years ago 
This a variation on issue #94.

Original comment by kim.gras...@gmail.com on 3 Aug 2014 at 8:40