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