<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Apr 13, 2009, at 1:50 PM, Daniel Dunbar wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Sun, Apr 12, 2009 at 1:43 AM, Chris Lattner <span dir="ltr"><<a href="mailto:sabre@nondot.org">sabre@nondot.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"> Author: lattner<br> Date: Sun Apr 12 03:43:13 2009<br> New Revision: 68894<br> <br> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=68894&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=68894&view=rev</a><br> Log:<br> Fix <a href="rdar://6771034">rdar://6771034</a>: don't warn on use of forward declared protocol in protocol<br> list of another protocol definition. This warning is very noisy and GCC doesn't<br> produce it so existing code doesn't expect it.</blockquote><div><br></div><div>Ok, seems like a necessary concession. However, this construct is also somewhat questionable and at some point we should move back towards having the warning enabled. Can we retain a -W.... for this?</div></div></blockquote><div><br class="webkit-block-placeholder"></div>I think the best place for the warning is when protocol is actually used in a class (or in a qualified type maybe):</div><div><br class="webkit-block-placeholder"></div><div>@protocol P1</div><div><br class="webkit-block-placeholder"></div><div>@protocol P2<P1> @end</div><div><br class="webkit-block-placeholder"></div><div>@interface I <P2> // warn here</div><div>@end</div><div><br class="webkit-block-placeholder"></div><div>I am not sure though, how noisy this one will be.</div><div><br class="webkit-block-placeholder"></div><div>- Fariborz</div><div><br><blockquote type="cite"><div class="gmail_quote"> <div><br></div></div></blockquote></div></body></html>