[petsc-dev] pushing MASSIVE change to PETSc-dev

[Barry Smith]

On Apr 21, 2011, at 10:18 AM, Jed Brown wrote:

> One consequence of this change is that, in general, it should no longer be necessary to use the construct
> 
> if (x->y) {ierr = YDestroy(x->y);CHKERRQ(ierr)}
> 
> Instead, you should be able to always use
> 
> ierr = YDestroy(&x->y);CHKERRQ(ierr);
> 
> There are still some of the former around, but it should not be used in new code.

   Correct, I removed all that I found but may have missed a few. Also you need not do VecDestroy(x->y); x->y = 0; since the destroy handles that automatically. I will make another sweep through the code to try to eliminate the unneeded checks.

    Barry