On Wed, Jun 10, 2009 at 11:33 AM, Jed Brown <span dir="ltr"><<a href="mailto:jed@59a2.org">jed@59a2.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Barry Smith wrote:<br>
><br>
>    Go ahead. This has been on the todo list for a decade.<br>
<br>
I see two ways to do it.  One is to define a PCOperation enum and have<br>
PCShellSetOperation (analogous to MatShell), the other is to just update<br>
the current interface for void* -> PC.  The former results in less code<br>
(though more changes today), but the latter offers stronger type<br>
checking and is not outrageously bloated because the PC interface is<br>
pretty small.  What is your preference?</blockquote><div><br>I like the latter.  I never liked the enums, but Mat has so many operations.<br><br>  Matt<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
>    Remember to update dev.html with the change and change any examples<br>
> that use PCShell.<br>
><br>
>    Note you have to update the Fortran stubs for these beasts (some of<br>
> the uglier stuff in PETSc.)<br>
<br>
Yep.<br>
<font color="#888888"><br>
Jed<br>
<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener<br>