<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Nov 14, 2013 at 2:01 PM, Geoffrey Irving <span dir="ltr"><<a href="mailto:irving@naml.us" target="_blank">irving@naml.us</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The SetupElement, SetupMaterialElement, and SetupBdElement functions<br>
are almost entirely duplicates of each other:<br>
<br>
    ./src/snes/examples/tutorials/ex12.c<br>
<br>
This makes it a bit difficult to understand what's going on: I'm new<br>
the code, so I had to run a diff to confirm that the boundary spaces<br>
are allocated identically to the interior spaces (I naively expected a<br>
difference, but it's apparently elsewhere in the code).<br>
<br>
Are there any objections to me refactoring the code a bit to avoid the<br>
duplication?</blockquote><div><br></div><div>No, however it would be nice to verify that the current tests run:</div><div><br></div><div>  ./config/builder2.py check src/snes/examples/tutorials/ex12.c</div><div><br></div>
<div>The copying was intentional, since I never refactor the first time I am</div><div>writing things. Now that it has started to work, that is appropriate.</div><div><br></div><div>  Thanks,</div><div><br></div><div>     Matt</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
Geoffrey<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <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
</div></div>