<html>#2337: AHFinderDirect does not mark surfaces as "invalid" after it stops looking for them
<table style='border-spacing: 1ex 0pt; '>
<tr><td style='text-align:right'> Reporter:</td><td>Roland Haas</td></tr>
<tr><td style='text-align:right'>   Status:</td><td>open</td></tr>
<tr><td style='text-align:right'>Milestone:</td><td></td></tr>
<tr><td style='text-align:right'>  Version:</td><td></td></tr>
<tr><td style='text-align:right'>     Type:</td><td>bug</td></tr>
<tr><td style='text-align:right'> Priority:</td><td>major</td></tr>
<tr><td style='text-align:right'>Component:</td><td>EinsteinToolkit thorn</td></tr>
</table>

<p>Comment (by Roland Haas):</p>
<p>On closer inspecting this may be correct, but benefit from a comment. Namely, not having a <code>return</code> statement in 678 means that (due to the <code>assert</code> in 677) the code is guaranteed to clear <code>sf_valid</code> in 683 and return there. </p>
<p>Ie the code will mark the surface that was not looked for as invalid.</p>
<p>--<br/>
Ticket URL: <a href='https://bitbucket.org/einsteintoolkit/tickets/issues/2337/ahfinderdirect-does-not-mark-surfaces-as'>https://bitbucket.org/einsteintoolkit/tickets/issues/2337/ahfinderdirect-does-not-mark-surfaces-as</a></p>
</html>