<div dir="ltr"><div class="gmail_extra">On Mon, Jan 14, 2013 at 11:17 AM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_quote">
<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 id=":ke"> Note that even if you change it to an inline function it still will not give the correct stack frame since it is not inside the PetscFunctionBegin/End.<br>
</div></blockquote><div><br></div><div style>The point is that an inline function routine couldn't be called in a declaration, so it would force people to move it. You're right that it looks like it was only used incorrectly in the places fixed here.</div>
<div style><br></div><div style><a href="https://bitbucket.org/petsc/petsc-dev/commits/59c614dd5ce57f9497e794f6495fc91990a89274">https://bitbucket.org/petsc/petsc-dev/commits/59c614dd5ce57f9497e794f6495fc91990a89274</a><br>
</div><div> </div><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 id=":ke">
<br>
  Shouldn't we simply ban its use in the declarations part of the routines? </div></blockquote><div><br></div><div style>I thought it was already banned and that nobody was paying attention, sort of like the // comments.</div>
<div> </div><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 id=":ke">Note that its use in, for example,<br>

<br>
    } else {<br>
      PetscBLASInt ione = 1;<br>
      PetscScalar aone = 1.0, azero = 0.0;<br>
      PetscBLASInt neqs = PetscBLASIntCast(bcgsl->ell-1);<br>
<br>
is legal since it is within a PetscBegin/Return block and the last declaration given.<br>
<br>
A quick etags search found that it is being used correctly already almost anywhere. The suspicious place is only in the documentation :-)<br>
<br>
Thus I say, fix the documentation and check that it is never used incorrectly but don't change its form.</div></blockquote></div><br><br></div></div>