<div class="gmail_quote">On Thu, May 10, 2012 at 12:56 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">
I'm new to working with the clang source and wanted to contribute, so I tried to pick a bug that seemed fairly straightforward.<br>
This patch adds the enhancement requested in <a href="http://llvm.org/bugs/show_bug.cgi?id=11857" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=11857</a><br>
<br>
Let me know if I've done something unforgivable in my patch, or any tips/suggestions.<br>
Since I'm still new to the codebase, I would be open to suggestions for easy bugs that would increase my familiarity.<br></blockquote></div><br><div>The patch generally looks good, thanks! Presumably this only applies to the case where there are no arguments, because otherwise we couldn't know /which/ argument was missing?</div>
<div><br></div><div>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().</div>
<div><br></div><div>It would also be great to extend this to the 'candidate function not viable' diagnostics in C++.</div>