> On May 15, 2016, at 1:04 PM, Munson, Todd <[email protected]> wrote: > > > I would argue that they should only be members for SNESVI.
The problem with that approach is that if someone calls SNESVISetXXX() but uses NEWTONLS then the bounds would be ignored while the user likely expects them to be used and hence gets upset that they get the "wrong" answer. This is why we ended up putting error checks in all the methods that do not use bounds. Barry > > Todd. > >> On May 14, 2016, at 10:46 PM, Matthew Knepley <[email protected]> wrote: >> >> If I call this on a regular SNES, like newtonls, I get a memory leak >> because it defaults to the VI version, but SNESReset() only uses >> the newtonls version. We should decide whether these are members >> of SNES itself or SNESVI. >> >> Matt >> >> -- >> What most experimenters take for granted before they begin their experiments >> is infinitely more interesting than any results to which their experiments >> lead. >> -- Norbert Wiener >
