On Tue, Aug 21, 2012 at 11:24 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
On Aug 21, 2012, at 10:53 AM, Jed Brown <<a href="mailto:jedbrown@mcs.anl.gov">jedbrown@mcs.anl.gov</a>> wrote:<br>
<br>
> On Tue, Aug 21, 2012 at 10:39 AM, Matthew Knepley <<a href="mailto:knepley@ci.uchicago.edu">knepley@ci.uchicago.edu</a>> wrote:<br>
> Wolfgang is not wrong here. I was against the name change :)<br>
><br>
> I complained about this in May when it was pushed.<br>
<br>
</div>    Then who the fuck changed this. It sure as hell wasn't me.</blockquote><div><br></div><div>I believe Hong.</div><div><br></div><div>   Matt</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
    Barry<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
><br>
><br>
>    Matt<br>
><br>
> ---------- Forwarded message ----------<br>
> From: Wolfgang Bangerth <<a href="mailto:bangerth@math.tamu.edu">bangerth@math.tamu.edu</a>><br>
> Date: Mon, Aug 20, 2012 at 10:30 PM<br>
> Subject: Changes in PETSc<br>
> To: Matthew Knepley <<a href="mailto:knepley@ci.uchicago.edu">knepley@ci.uchicago.edu</a>>, "Toby D. Young" <<a href="mailto:tyoung@ippt.gov.pl">tyoung@ippt.gov.pl</a>><br>
><br>
><br>
><br>
> Hi Matt,<br>
> I ran across something today that I believe perfectly illustrates the kind of thing that drives everyone completely mad about PETSc's habit of changing everything in every release. I was staring for a long time at a patch by one of our contributors labeled "fix catastrophic typo" that reads like this:<br>

> ...........................<br>
> -    int ierr = KSPSetType (ksp, const_cast<char *>(KSPCHEBYSHEV));<br>
> +                                     // set the type of solver.<br>
> +    int ierr;<br>
> +<br>
> +#ifdef DEAL_II_PETSC_VERSION_LT(3,3,0)<br>
> +    ierr = KSPSetType (ksp, const_cast<char *>(KSPCHEBYCHEV));<br>
><br>
> They can get rid of the const_cast.<br>
><br>
> Also, instead of the magic version nonsense, they can write<br>
><br>
> #ifndef KSPCHEBYSHEV<br>
> #  define KSPCHEBYSHEV KSPCHEBYCHEV<br>
> #endif<br>
><br>
> and then always use the new version in their code.<br>
><br>
> Also, a package like deal.II would be better off snooping the list of valid parameters using PetscFListGet() so that they would *automatically* support *all* KSPs and PCs in PETSc.<br>
><br>
> +#else<br>
> +    ierr = KSPSetType (ksp, const_cast<char *>(KSPCHEBYSHEV));<br>
> +#endif<br>
> ...........................<br>
> It took me minutes to find the difference: before 3.3, the name was spelled chebyChev, after that chebyShev. That's really a completely unnecessary change -- but hugely disruptive because not compatible, incredible difficult to see, and it also has no real value at all: whether you want to spell the name the French or the American way really makes no difference at all. Worst of all, it would have been absolutely trivial to make the change backward compatible by simply doing<br>

>   #define KSPCHEBYSHEV KSPCHEBYCHEV<br>
> but that didn't happen.<br>
><br>
> So why do it? Sorry for venting, but it just doesn't make sense to me...<br>
> Best<br>
>  W.<br>
><br>
> --<br>
> ------------------------------------------------------------------------<br>
> Wolfgang Bangerth               email:            <a href="mailto:bangerth@math.tamu.edu">bangerth@math.tamu.edu</a><br>
>                                 www: <a href="http://www.math.tamu.edu/~bangerth/" target="_blank">http://www.math.tamu.edu/~bangerth/</a><br>
><br>
><br>
><br>
><br>
> --<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<br>
><br>
<br>
</div></div></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<br>