r220703 - c++11 patch to issue warning on missing 'override' on

jahanian fjahanian at apple.com
Tue Oct 28 15:03:56 PDT 2014


> On Oct 28, 2014, at 2:56 PM, Nico Weber <thakis at chromium.org> wrote:
> 
> Cool!
> 
> Would it be possible to emit a fixit for inserting override for the cases where the method isn't from a macro expansion? That would make it easy to automatically insert override everywhere it's missing. (I think there's some function somewhere already to find of "override" is usually spelled "OVERRIDE" in the codebase, in case the code's using override through a macro.)
> 

Yes. But, I haven’t figure out all the possible cases where ‘override’ location  can be obtained (my very first patch had a fixit option; but it was deemed insufficient and not covering all cases).

- Fariborz





More information about the cfe-commits mailing list