<div class="gmail_quote">On Wed, Aug 15, 2012 at 1:29 PM, jahanian <span dir="ltr"><<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><br><div><div><div class="h5"><div>On Aug 15, 2012, at 1:24 PM, Jean-Daniel Dupas wrote:</div><br><blockquote type="cite"><div style="word-wrap:break-word"><br><div><div>Le 15 août 2012 à 21:10, jahanian <<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@apple.com</a>> a écrit :</div>
<br><blockquote type="cite"><div style="word-wrap:break-word"><br><div><div>On Aug 15, 2012, at 12:04 PM, Jordan Rose wrote:</div><br><blockquote type="cite"><div style="word-wrap:break-word"><br><div><div>On Aug 15, 2012, at 11:58 , jahanian <<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@apple.com</a>> wrote:</div>
<br><blockquote type="cite"><div style="word-wrap:break-word"><br><div><div>On Aug 15, 2012, at 11:49 AM, Jordan Rose wrote:</div><br><blockquote type="cite"><div style="word-wrap:break-word"><div>What <i>is</i> the unexpected behavior? Or what will it <i>not</i> do that I would expect it to do? (I forget what __private_extern__ is actually for.)</div>
<br><div>On Aug 15, 2012, at 11:42 , Fariborz Jahanian <<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@apple.com</a>> wrote:</div></div></blockquote></div><br><div>Depending on __private_extern__ decl. being in header file or main file, visibility attribute may or may not show up in the</div>
<div>generated code (and it may not be a tentative definition). See the radar for why Nick thinks this warning is needed to</div><div>get us out of the jam for this gcc compatible behavior.</div></div></blockquote><br></div>
<div>I think what I was trying to say is that the warning should be a little more explanatory. I don't have an alternative wording myself, though.</div></div></blockquote><br></div><div>I tried to make the warning as long and explanatory as possible. But, user may have to read the radar's long description to see</div>
<div>the rational  for the warning. But, feel free if you can improve on the message.</div></div></blockquote><div><br></div><div>Would be fine if radar where visible by the user. Unfortunately, this is not the case…</div>
</div></div></blockquote><div><br></div></div></div>Oops. I was referring to Jordan, who has access to radars. A more detailed explanation can go into the man page (is there one for clang?).</div></div></blockquote><div><br>
</div><div>The diagnostic message should be sufficient for a typical user to have some idea of what is wrong. The current diagnostic does not meet this standard. If you explain in a bit more detail what the problem is, maybe someone will be able to come up with better diagnostic wording. In any case, diagnostic messages should not start with a capital letter.</div>
</div>