bugfix for latest version of vn

Bob Mcqueer bobm at rtech.UUCP
Wed Jan 14 04:39:21 AEST 1987


The continuation of header lines has recently been brought to my attention.
I've never seen it before, so I assume it is a relatively recent addition.
Yes, indeed, there is a problem if the "Lines:" header line can't be found,
and a protection against division by 0 should be added in any case.

Andrew's patch looks like the right idea.  Thanks.  I'll check it out for
the "official" copy.

You might also change the decision from "line contains ':'" to "line is
empty".  This is a quick fix which remedies the problems being discussed,
although only the first line of a multi-line header will be found.
I hadn't originally used this rule because I was worried about chopping
off lines of articles if the empty line was missing.

When I saw Andrew's article this morning, I was going to publish the
following change in reader.c:

205c205,217
< 				percent = ((Rlines-Hlines-2)*100)/artlin;
---
> 			{
> 				/*
> 				** protect against division by zero - artlin
> 				** shouldn't actually come up zero at this
> 				** point if "Lines:" header line was found
> 				** and was correct.  "999" will let user
> 				** know the percentage is obviously wrong.
> 				*/
> 				if (artlin > 0)
> 					percent = ((Rlines-Hlines-2)*100)/artlin;
> 				else
> 					percent = 999;
> 			}
301c313
< 		if (index(buf,':') == NULL)
---
> 		if (buf[0] == '\n')
-- 

Bob McQueer
{amdahl, sun, mtxinu, hoptoad, cpsc6a}!rtech!bobm



More information about the Comp.sources.bugs mailing list