sarojvarma / include-what-you-use

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

README file refers to .cpp files in source but they're now .cc files. #84

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
In describing the function of each file, it uses a .cpp name instead of a .cc 
name.  A simple command fixed these: sed -i -r -e "s/iwyu([^ 
]*)\.cpp/iwyu\1.cc/g" README.txt

I've attached the patch.

Original issue reported on code.google.com by ryan.pav...@gmail.com on 20 Nov 2012 at 7:53

Attachments:

GoogleCodeExporter commented 9 years ago
Updated patch with one that includes a human change as well.

Original comment by ryan.pav...@gmail.com on 20 Nov 2012 at 8:30

Attachments:

GoogleCodeExporter commented 9 years ago
Thanks!

I wonder if the files should all be .cpp instead, though, to be more in line 
with LLVM/Clang?

Original comment by kim.gras...@gmail.com on 21 Nov 2012 at 6:14

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r378.

Original comment by vsap...@gmail.com on 25 Nov 2012 at 10:23

GoogleCodeExporter commented 9 years ago
Looks like at first files were .cpp, but then were renamed, probably according 
to Google standard. I don't support renaming files to .cpp because currently I 
see no reason to be compliant with LLVM/Clang that much.

Original comment by vsap...@gmail.com on 25 Nov 2012 at 10:36