<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Here is a new patch for warning on missing ‘override’ with following changes from the old patch.<div><br></div><div>1. I have incorporated few refactoring suggested by you and others.</div><div>2. I removed ‘FixIt’ for now. It is not necessary for this patch and something we can incorporate</div><div> later if this is something we want to do.</div><div>3. After reading all the discussions I no longer warn on missing ‘override’ on ‘final’ methods.</div><div> I see the point made that it is redundant for ‘final’ which already means ‘override. And since it is in</div><div> violation of coding guidelines, it is more likely to break code. I also understand</div><div> that it is a contentious issue which may be changed later.</div><div>4. This patch warns if overriding method (without ‘override' keyword) is explicitly declared as ‘virtual’. </div><div> I feel that this is a useful warning as ‘override’ keyword makes user’s intention verbose. But, I</div><div> can be convinced otherwise. </div><div><br></div><div> - Please review.</div><div><br></div><div>- Fariborz</div><div><br></div><div><br></div><div> </div></body></html>