[cfe-commits] Patch: Change how struct/class mismatches are handled

Richard Trieu rtrieu at google.com
Sun May 29 13:51:14 PDT 2011


Made changes to how 'struct'/'class' mismatches are handled.
- Removed fix-it hints from template instaniations since changes to the
templates are rarely helpful.
- Changed the caret in template instaniations from the class/struct name to the
class/struct keyword, matching the other warnings.
- Do not offer fix-it hints when multiple declarations disagree.  Warnings are
still given.
- Once a definition is found, offer a fix-it hint to all previous declarations
with wrong tag.
- Declarations that disagree with a previous definition will get a fix-it hint
to change the declaration.

Patch is attached and available at:
http://codereview.appspot.com/4517115/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110529/7c84c814/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: struct-class-mismatch.patch
Type: text/x-patch
Size: 15018 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110529/7c84c814/attachment.bin>


More information about the cfe-commits mailing list