search

doctaweeks / include-what-you-use

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

fix_include --safe_headers doesn't work and will delete header lines #180

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The line 

if not (flags.safe_headers and not _MayBeHeaderFile(iwyu_record.filename)):

should actually be

if not flags.safe_headers and not _MayBeHeaderFile(iwyu_record.filename):

Otherwise the script will still delete header lines even using 

fix_include --nocomments --safe_headers

Original issue reported on code.google.com by conc...@web.de on 21 Mar 2015 at 10:44

GoogleCodeExporter commented 9 years ago 
Ok, I misunderstood this thing. I thought this will avoid removing header 
includes but it is only to not remove includes from headers.

Please close

Original comment by conc...@web.de on 21 Mar 2015 at 10:46

GoogleCodeExporter commented 9 years ago 
OK, thanks!

Original comment by kim.gras...@gmail.com on 21 Mar 2015 at 12:57