note
Athanasius
<p>Can’t speak for [jdporter], but to my way of thinking the loops (C-style instead of foreach-style) are the least of the problems. To <i>my</i> old eyes the major issues are:</p>
<ol>
<li><p><c>$mode</c> never changes, but it’s re-tested each time through the main loop.</p></li>
<li><p>The code is monolithic and should be refactored into subroutines (one for each mode).</p></li>
<li><p><c>use warnings;</c> and <c>use strict;</c> are missing, and all the variables are global.</p></li>
</ol>
<p>Hope that helps,</p>
<div class="pmsig"><div class="pmsig-968231">
<p>
<table width="100%">
<tr>
<td align="left">
Athanasius <font color="#008000"><</font>[href://http://www.biblegateway.com/passage/?search=John%203:16&version=NLV|<font color="#008000">°</font>]<font color="#008000">(((><</font> <i>contra mundum</i>
</td>
<td align="right">
[href://http://translate.google.com.au/#la/en/Iustus%20alius%20egestas%20vitae%2C%20eros%20Piratica%2C|<b>Iustus alius egestas vitae, eros Piratica,</b>]
</td>
</tr>
</table>
</p>
</div></div>
1016588
1016953