
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">

<HTML>

<HEAD>

<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">

<META NAME="Generator" CONTENT="MS Exchange Server version 6.5.7650.28">

<TITLE>LRUD bug in 1.1.9</TITLE>

</HEAD>

<BODY>

<!-- Converted from text/plain format -->


<P><FONT SIZE=2>Hi Olly,<BR>

<BR>

I'm afraid I've got bad news yet again.<BR>

<BR>

I just got round to downloading 1.1.9. Looks like you introduced a new bug<BR>

while fixing the old bug in img.c. At line 823, it fails to reset<BR>

pimg-&gt;pending to 0, so whenever you open a 3d file with any LRUD data it goes<BR>

into an infinite loop after reading the first batch of LRUD data.<BR>

<BR>

823c823,827<BR>

&lt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (pimg-&gt;pending == 256) return img_XSECT_END;<BR>

---<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (pimg-&gt;pending == 256)<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; pimg-&gt;pending = 0;<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return img_XSECT_END;<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }<BR>

<BR>

Incidentally, while testing this I hacked up dump3d to output cross-section<BR>

data as well -- just a three-line change. I reckon it'd probably be sensible<BR>

to incorporate this into future versions. Here's the diff to dump3d.c:<BR>

<BR>

123a124,126<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; case img_XSECT:<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; printf(&quot;XSECT %f %f %f %f [%s]\n&quot;, pimg-&gt;l, pimg-&gt;r, pimg-&gt;u, pimg-&gt;d, pimg-&gt;label);<BR>

&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; break;<BR>

<BR>

<BR>

<BR>

<BR>

Dave<BR>

<BR>

--<BR>

David Loeffler<BR>

Mathematics Department<BR>

Imperial College, London<BR>

<BR>

</FONT>

</P>


</BODY>

</HTML>