On Thu, Mar 21, 2013 at 3:31 PM, Barry Smith <bsmith@mcs.anl.gov> wrote:

> Why isn't this `PetscOptionsList()` so that it shows up in `-help`?

My mistake, I cut and pasted the PetscOptionsGetInt() from the line above and figure that it mustn't be in a PetscOptionsBegin{} phase. You can fix it if you like.

Hmm, problem is deeper than that. If the method has no sub-solver, wouldn't we rather not be checking this option, so that it shows up as an unused option? (This is also useful if a PC logically ought to pay attention to this option, but isn't.) What about making PCSetUseAmat() use PetscTryMethod() and move the check of "-pc_use_amat" into each implementation?

Do we really want to leave the confusing option in place for all the PCs that don't have inner solvers?