<div dir="ltr"><div><div><div>Barry,<br><br></div>The branch resolves the difficulties. Sorry for the sloppy code, it was just a quick fix to illustrate something. Had I known you were going to use it as a patch I would have tested it more <span>thoroughly.<br><br></span></div><span>Thanks,<br></span></div><span>Sander<br></span></div><div class="gmail_extra"><br><div class="gmail_quote">On 26 November 2016 at 06:02, 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>
   Sander,<br>
<br>
    Thanks for the patch, I have put it into the branch barry/Sander/fix-pcreset-<wbr>fieldsplit and confirmed that it does<br>
support reset (after fixing some bugs) and merged it into next for testing. Can you try it and see if it resolves the difficulties you were having?<br>
<br>
   Barry<br>
<br>
Before submitting patches you should try the test codes with -malloc_test or under valgrind, you would have found some memory leaks due to bugs in your patch and from those found the bugs in the patch you sent.<br>
<div><div class="h5"><br>
<br>
> On Nov 22, 2016, at 3:10 AM, Sander Arens <Sander.Arens@UGent.be> wrote:<br>
><br>
> It works now for the particular case when there's a SNESReset before a SNESSolve, but when you put a SNESReset and SNESSolve again after the first SNESSolve it breaks again (I noticed this first  playing around with TSArkimex when it broke at this line).<br>
><br>
> Destroying the fieldsplit linked list in PCReset_FieldSplit seemed to work (see diff file)...<br>
><br>
> On 21 November 2016 at 21:02, Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> wrote:<br>
> Okay, try out the fix in next.<br>
><br>
>   Thanks,<br>
><br>
>     Matt<br>
><br>
> On Thu, Nov 17, 2016 at 3:11 PM, Matthew Knepley <<a href="mailto:knepley@gmail.com">knepley@gmail.com</a>> wrote:<br>
> On Thu, Nov 17, 2016 at 1:22 PM, Sander Arens <<a href="mailto:Sander.Arens@ugent.be">Sander.Arens@ugent.be</a>> wrote:<br>
> I think there's a bug in PCSetUp_FieldSplit or PCReset_FieldSplit. I can get this by putting a SNESReset before the SNESSolve in snes ex77 and running ./config/builder2.py check src/snes/examples/tutorials/<wbr>ex77.c. Can somebody reproduce this?<br>
><br>
> You are right. Fixing...<br>
><br>
>    Matt<br>
><br>
> Thanks,<br>
> Sander<br>
><br>
><br>
><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>
><br>
><br>
><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>
><br>
</div></div>> <fieldsplit.diff><br>
<br>
</blockquote></div><br></div>