Matt said he was afraid of your wrath because we discussed doing that a long time ago and you were opposed. I'd rather have it in the Vec and Mat data structures, if nothing else than because it removes the string lookups.<div>
<br></div><div>Note that the slots in Vec and Mat should be a PetscObject because their destructor needs to be called. Also note that we don't automatically check those slots for circular references so we need to think about the destruction process to figure out if we still need to prevent that loop and if so, how to check.<br>
<div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 20, 2012 at 10:47 PM, 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>
On Nov 20, 2012, at 2:30 PM, Matt Knepley <<a href="mailto:notifications-noreply@bitbucket.org">notifications-noreply@bitbucket.org</a>> wrote:<br>
<br>
> New comment on changeset:<br>
><br>
> <a href="https://bitbucket.org/petsc/petsc-dev/changeset/6e9d4e780b4d4f1b06b023485bbcf786a79fb6fe#comment-70978" target="_blank">https://bitbucket.org/petsc/petsc-dev/changeset/6e9d4e780b4d4f1b06b023485bbcf786a79fb6fe#comment-70978</a><br>
><br>
> Matt Knepley (knepley) said:<br>
><br>
> What the heck are those calls doing in examples? The user should NEVER NEVER NEVER be doing that. There are accessors for that.<br>
<br>
There is an accessor for doing<br>
<br>
ierr = PetscObjectCompose((PetscObject)J,"__PETSc_dm",(PetscObject)PETSC_NULL);CHKERRQ(ierr);<br>
<br>
?<br>
<br>
BTW: if you are going to write accessors like MatSetDM() then why go through the PetscObjectCompose() nonsense? Why not just stick the dm into the mat data structure as a void * in your accessor?<br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
<br>
><br>
> --<br>
> This is an comment notification from <a href="http://bitbucket.org" target="_blank">bitbucket.org</a>.<br>
> You are receiving this because you are participating<br>
> in a comment thread.<br>
<br>
</font></span></blockquote></div><br></div></div>