<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">Ah I see. That's because the last argument is a C++ default argument. It looks like the convention in this file is that the error argument should be the last non-defaulted argument.</span></blockquote><div> </div><div>I think it should be:</div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px">void StepOver(lldb::RunMode stop_other_threads, lldb::SBError &error); // no default argument!<br><br></blockquote>Ie. if you want the overload with error you need to pass RunMode explicitly. Keeping the overload set manageable is a good practice in general, not just because of SWIG (default arguments + overloaded arguments can easily get out of hand)<div><div><br></div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 11, 2018 at 5:51 PM, Adrian Prantl via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aprantl accepted this revision.<br>
aprantl added a comment.<br>
<br>
Ah I see. That's because the last argument is a C++ default argument. It looks like the convention in this file is that the error argument should be the last non-defaulted argument.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D47991" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D47991</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>