r218925 - Patch to warn if 'override' is missing

[jahanian]

On Oct 3, 2014, at 10:29 AM, Richard Smith <richard at metafoo.co.uk> wrote:

> > But a bigger question is if there is objection to having this kind of potentially noisy warning? There will be projects which break and patch gets reverted.
> 
> I think it's fine if it's done right. I don't think the current approach is viable.
> 
> 
I don’t quite understand “current approach". You mean this particular warning after ‘final’  you are objecting to? or reverting a patch because of new warnings.
- Fariborz
> >
> > - Fariborz
> >
> >
> >

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141003/6d070a9c/attachment.html>