<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Apr 27, 2016 at 11:52 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Wed, Apr 27, 2016 at 11:50 AM, Stephan T. Lavavej via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">[Dmitry Polukhin]<br>
<span>> As for 'A*' to 'B*" case, indeed it is not very common and it looks like someone just forgot to specify 'public'.<br>
<br>
</span>If we're doing something gratuitously nonconformant in MS product code or even sample code, you should report it to us, so we can fix it upstream, instead of having to add more bug compatibility to clang, or tolerate known failures when using clang on MS sample code.<br>
<br>
I don't have the context here (just subscribed to cfe-commits), but if you can send me a self-contained writeup of where and what the problem is in the WTL sample, I can go find the owner and report it to them. We can't go back in time and fix released stuff, but we should be able to apply simple fixes to future releases of products and samples.<br></blockquote><div><br></div></span><div>You can find a description of the problem in <a href="http://llvm.org/PR27337" target="_blank">llvm.org/PR27337</a></div></div></div></div></blockquote><div><br></div><div>Brief summary:</div><div><br></div><div>The WTL bug is the missing 'public' on the second base class on this line: <a href="https://sourceforge.net/p/wtl/code/HEAD/tree/trunk/wtl/Samples/MDIDocVw/mainfrm.h#l636">https://sourceforge.net/p/wtl/code/HEAD/tree/trunk/wtl/Samples/MDIDocVw/mainfrm.h#l636</a></div><div><br></div><div>The C1xx bug is that it accepts a (non-C-style) cast from a base class to an inaccessible derived class.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Thanks,<br>
Stephan T. Lavavej<br>
Senior Developer - Visual C++ Libraries<br>
<div><div>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></span></div><br></div></div>
</blockquote></div><br></div></div>