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
