<div dir="ltr">On Tue, Apr 16, 2013 at 6:31 AM, Jed Brown <span dir="ltr"><<a href="mailto:jedbrown@mcs.anl.gov" target="_blank">jedbrown@mcs.anl.gov</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> writes:<br>
<br>
> On Mon, Apr 15, 2013 at 8:13 PM, Jed Brown <<a href="mailto:jedbrown@mcs.anl.gov">jedbrown@mcs.anl.gov</a>> wrote:<br>
><br>
>> Should we change the names below too?  I guess I didn't see value in the<br>
>> names above (basically Hungarian notation for functions) and I don't<br>
>> think they make anything more consistent.<br>
>><br>
><br>
> You are suggesting getting rid of the class prefix? We do consistently use<br>
> this<br>
> when we have different input types. I don't think it cleans up the code<br>
> enough to<br>
> change.<br>
<br>
Well, TSGetTSAdapt used to be named TSGetAdapt.  It was changed in:<br>
<br>
  commit ad6bc4219076b084d4a49a29c04a77944f238f10<br>
  Author: Barry Smith <<a href="mailto:bsmith@mcs.anl.gov">bsmith@mcs.anl.gov</a>><br>
  Date:   Sun Dec 9 18:14:24 2012 -0600<br>
<br>
      My first TS restart just 8 million other changes needed PETSc!<br>
      TSGetAdapt() changed to TSGetTSAdapt() because it gets a TSAdapt object, not an Adapt object<br>
      added TSAdapt binary view and load<br>
<br>
      Hg-commit: c5ab7de6c480df2d7b84dc349ac7871cdc85dcbe<br>
<br>
But this rationale does not match the more established convention used<br>
by functions such as those below.  We still haven't released, so we<br>
could switch back to TSGetAdapt without people hating us.<br>
<br>
Note that SNESGetSNESLineSearch was briefly named SNESGetPetscLineSearch<br>
(when it returned a PetscLineSearch), which doesn't match any existing<br>
convention.<br>
<br>
If we consider SNESGetSNESLineSearch to be more of an outlier than those<br>
below, I would rather at least change TSGetTSAdapt back to TSGetAdapt<br>
(i.e., no change for people following released PETSc) and perhaps also<br>
change to SNESGetLineSearch.  If we say TSGetTSAdapt is the proper way<br>
to name functions, we're obligated to apply that rule consistently,<br>
which means quite a lot of changes for users (the list below is not<br>
exhaustive).<br></blockquote><div><br></div><div style>Okay, that makes sense. I agree.</div><div style><br></div><div style>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

>> MatGetMatInfo<br>
>> MatGetISColoring<br>
>> MatGetMatNullSpace<br>
>> SNESGetSNESConvergedReason<br>
>> MatGetISLocalToGlobalMapping<br>
>> MatGetISLocalToGlobalMappingBlock (oops, 33 characters)<br>
>> MatGetOrderingIS (?)<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
-- Norbert Wiener
</div></div>