<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 15, 2014 at 11:43 AM, jahanian <span dir="ltr"><<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@apple.com</a>></span> wrote:<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 style="word-wrap:break-word"><br><div><span class=""><div>On Dec 12, 2014, at 3:34 PM, Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:</div><br><blockquote type="cite"><div dir="ltr">Ah, I think I found it. You indented it like this:<div><br></div><div><div>      if (Method) {</div><div>        if (ObjCMethodDecl *BestMethod =</div><div>              SelectBestMethod(Sel, ArgsIn, Method->isInstanceMethod()))</div><div>          Method = BestMethod;</div><div>          SmallVector<ObjCMethodDecl*, 4> Methods;</div><div>          if (!CollectMultipleMethodsInGlobalPool(Sel, Methods, Method->isInstanceMethod()))</div><div>            DiagnoseUseOfDecl(Method, SelLoc);</div><div>      }</div><div>    }</div></div><div><br></div><div>but clang-format tells you that it should be indented like so:</div><div><br></div><div><div>      if (Method) {</div><div>        if (ObjCMethodDecl *BestMethod =</div><div>                SelectBestMethod(Sel, ArgsIn, Method->isInstanceMethod()))</div><div>          Method = BestMethod;</div><div>        SmallVector<ObjCMethodDecl *, 4> Methods;</div><div>        if (!CollectMultipleMethodsInGlobalPool(Sel, Methods,</div><div>                                                Method->isInstanceMethod()))</div><div>          DiagnoseUseOfDecl(Method, SelLoc);</div><div>      }</div><div>    }</div></div><div><br></div><div>That is, you added the {} to the outer if, but your indentation looks like you mentioned to add them to the inner if. (I should dust off and check in my -Windent warning.)</div></div></blockquote><div><br></div></span>My intention was as I coded. Have you looked at the warning and see if it is indeed calling an ‘unavailable’ method which previously went undiagnosed? We have uncovered</div><div>several other cases of making such calls.</div></div></blockquote><div><br></div><div>Let's continue the discussion on <a href="http://llvm.org/bugs/show_bug.cgi?id=21587" target="_blank" style="font-size:13px">http://llvm.org/bugs/show_bug.cgi?id=21587</a> :-) This does look like a false positive, and it only happens with precompiled headers (which shouldn't change behavior, but does). </div></div></div></div>