<div>Did we just adopt a new line formatting convention and nobody told me?</div><div><br></div><div><div><font face="courier new, monospace">          ierr = PetscObjectGetOptionsPrefix((PetscObject)(ilink->ksp),&prefix); CHKERRQ(ierr);</font></div>
<div><font face="courier new, monospace">          ierr = PetscObjectSetOptionsPrefix((PetscObject)(dms[i]), prefix);     CHKERRQ(ierr);</font></div><div><font face="courier new, monospace">          ierr = KSPSetDM(ilink->ksp, dms[i]);                                   CHKERRQ(ierr);</font></div>
<div><font face="courier new, monospace">          ierr = KSPSetDMActive(ilink->ksp, PETSC_FALSE);                        CHKERRQ(ierr);</font></div></div><div><font face="courier new, monospace">          ierr = PetscObjectIncrementTabLevel((PetscObject)dms[i],(PetscObject)ilink->ksp,0); CHKERRQ(ierr);</font></div>
<div><br></div><div><div><font face="courier new, monospace">        }</font></div><div><font face="courier new, monospace">        else {</font></div></div><div><div><br></div><div><font face="courier new, monospace">        if(jac->reset)  </font></div>
<div><font face="courier new, monospace">          SETERRQ(((PetscObject)pc)->comm,PETSC_ERR_SUP,"Cases not yet handled when PCReset() was used");</font></div></div><div><br></div><div>Knowing Barry, he's going to fix this formatting the next time he runs across that file just to make it consistent (at the unfortunate expense of making "hg annotate" less useful until the tools get better about tracing backward through history).</div>
<div><br></div><div>Is the tab level above even correct? Why would the KSP tab level be expected to match the DM tab level?</div><div><br></div><div>Do we need this new set of KSP-specific functions for manipulating tab levels? There isn't a KSPTypeCompare(), etc. I thought the convention within PETSc was that user-level APIs were usually exposed as type-specific routines, but developer-level stuff used PetscObject directly.</div>
<div><br></div>