<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 26, 2013, at 3:31 PM, Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Sat, Apr 27, 2013 at 1:24 AM, jahanian <<a href="mailto:fjahanian@apple.com">fjahanian@apple.com</a>> wrote:<br><blockquote type="cite">Why does this test use -ast-dump?  Usually c-index-test is used to<br>test if comment is attached to something.  There are fewer CHECK lines<br>that way.<br><br><br>OK. I will add a c-index test (without removing the existing one which took<br>quite time<br>to get it right :).<br></blockquote><br>That's exactly why c-index-test tests are preferred: they are more<br>maintainable.  So if you convert this file to c-index-test, just<br>remove the original test.<br></div></blockquote>Yes.<br><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br></div></blockquote><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br>If you decide that you really need to go this way (I don't think this<br>is a good approach, but anyway), please limit this to public<br></div></blockquote>This meets our user needs. But, limiting it to public inheritance</div><div>makes sense. In r180646.</div><div>- Fariborz</div><div><br><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">inheritance, which models is-a relationship.  It is semantically not<br>appropriate to check private and protected bases for documentation.<br>But private inheritance is not used much in C++, so this exclusion<br>will not help much to reduce confusion.<br><br>Dmitri<br><br>--<br>main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>(j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>*/</div></blockquote></div><br></body></html>