<div class="gmail_quote">On Thu, May 10, 2012 at 4:00 PM, Terry Long <span dir="ltr"><<a href="mailto:macmantrl@me.com" target="_blank">macmantrl@me.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">> The patch generally looks good, thanks!<br>
<br>
</div>Great, thanks for the feedback.<br>
<div class="im"><br>
> Presumably this only applies to the case where there are no arguments, because otherwise we couldn't know /which/ argument was missing?<br>
<br>
</div>Yes, only for the case where there are no arguments to a function that takes 1 argument. Almost impossible to determine the missing argument(s) otherwise.<br>
<div class="im"><br>> Please add test coverage for the err_typecheck_call_too_few_args_at_least_one diagnostic. Also, NamedDecl::getNameAsString is deprecated; please just use "<< FDecl->getParamDecl(0)", and use getParamDecl(0)->getDeclName()'s operator bool() in the test, rather than empty().<br>

<br>
</div>OK, I'll update this. I was using the online doxygen docs and didn't see any deprecation warnings. Anywhere where I can find that information?</blockquote><div><br></div><div>It's in include/clang/AST/Decl.h:138-141, though for some reason those comments aren't exposed to doxygen...</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> It would also be great to extend this to the 'candidate function not viable' diagnostics in C++.<br>

<br>
</div>I can take a look at this too.</blockquote><div><br></div><div>Awesome, thanks. </div></div>