<div>Hi Arjun,</div>
<div> </div>
<div>I tried this patch on llvm itself, it's too noisy.</div>
<div> </div>
<div>I reviewed some of them and i think it's not the right way to implement this.</div>
<div> </div>
<div>I think i have to cancle it.<br><br></div>
<div class="gmail_quote">在 2011年10月8日 下午1:06，章磊 <span dir="ltr"><<a href="mailto:ioripolo@gmail.com">ioripolo@gmail.com</a>></span>写道：<br>
<blockquote class="gmail_quote" style="PADDING-LEFT: 1ex; MARGIN: 0px 0px 0px 0.8ex; BORDER-LEFT: #ccc 1px solid">Hi Arjun,<br><br>I have not run this patch on some open source software, but i will do<br>it later. When i got any results, i will let you know asap.<br>
<br>2011/10/8, Arjun Singri <<a href="mailto:arjunsingri@gmail.com">arjunsingri@gmail.com</a>>:<br>
<div>
<div></div>
<div class="h5">> Have you run this on any open source software to see how many<br>> results/violations show up? If yes, sharing the results will be useful. If<br>> not, then its a good exercise to run it on an open source software. You can<br>
> even try running it on llvm itself.<br>><br>> Arjun<br>><br>> On Fri, Oct 7, 2011 at 7:12 AM, 章磊 <<a href="mailto:ioripolo@gmail.com">ioripolo@gmail.com</a>> wrote:<br>><br>>> Hi Clang,<br>
>><br>>> The attached patch adds a C++ Checker to clang, it checks whether<br>>> there are mistakes in override.<br>>><br>>> The mistake mean things like this:<br>>><br>>> class base {<br>
>>  virtual void foo() const {/*...*/}<br>>> };<br>>><br>>> class child: public base {<br>>>  /* child::foo is probably meant to override base::foo but type<br>>> signatures don't match<br>
>>  perfectly */<br>>>  void foo() {}<br>>> };<br>>><br>>> and this:<br>>><br>>> class base {<br>>>  void foo() const {}<br>>> };<br>>><br>>> class child: public base {<br>
>>  /* child::foo is probably meant to override base::foo but<br>>>  that function is not virtual. */<br>>>  void foo() const {}<br>>> };<br>>><br>>> I implement this as a static analysis checker, because i think it's<br>
>> not too visible and not too fast.<br>>><br>>> I will appreciate it if there are any advice about this patch.<br>>><br>>> --<br>>> Best regards!<br>>><br>>> Lei Zhang<br>>><br>
>> _______________________________________________<br>>> cfe-dev mailing list<br>>> <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
>><br>>><br>><br><br><br>--<br>Best regards!<br><br>Lei Zhang<br></div></div></blockquote></div><br><br clear="all"><br>-- <br>Best regards!<br><br>Lei Zhang<br>