Caught. I think I wrote the comment before I realized that it was better to match against the originaldm, especially since it's needed from callers like DMSNESSetFunction.<div><br></div><div><a href="https://bitbucket.org/petsc/petsc-dev/changeset/39c79c90e0f7a55f9466e52ce40c3a00f181db65">https://bitbucket.org/petsc/petsc-dev/changeset/39c79c90e0f7a55f9466e52ce40c3a00f181db65</a><br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Nov 25, 2012 at 4:37 AM, Barry Smith <span dir="ltr"><<a href="mailto:bsmith@mcs.anl.gov" target="_blank">bsmith@mcs.anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
   It takes some serious beer goggles to mix up SNES and DM (same problem with KSP and TS)<br>
<br>
<br>
<br>
  /* This is NOT reference counted. The SNES that originally created this context is cached here to implement copy-on-write.<br>
   * Fields in the SNESDM should only be written if the SNES matches originalsnes.<br>
   */<br>
  DM originaldm;<br>
<br>
<br>
</blockquote></div><br></div>