Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Posting-Version: version B 2.10.3 4.3bsd-beta 6/6/85; site sdcsvax.UUCP
Path: utzoo!watmath!clyde!burl!ulysses!bellcore!decvax!ittatc!dcdwest!sdcsvax!neil
From: neil@sdcsvax.UUCP (Neil Rhodes)
Newsgroups: net.unix-wizards
Subject: net.unix-wizards
Message-ID: <1429@sdcsvax.UUCP>
Date: Tue, 18-Feb-86 17:42:28 EST
Article-I.D.: sdcsvax.1429
Posted: Tue Feb 18 17:42:28 1986
Date-Received: Thu, 20-Feb-86 07:21:10 EST
Organization: EECS Dept. U.C. San Diego
Lines: 60
Keywords: bug report 4.3BSD spell deroff

I would imagine this bug is present in 4.2BSD as well
Note that this bug causes spell to only check words through the first
.PS < filename
in a file and ignores the rest.

It was real frustrating to find typos in my slides *after* having
run spell.

Subject: deroff can screws up on .PS < filename (pic construct
Index:	usr.bin/deroff.c 4.3BSD

Description:
	Note that .PS starts a pic macro.
	Pic allows
		.PS < filename
	to include filename (like .so) and in this case doesn't
	require a matching .PE
	Deroff shouldn't require a matching .PE in this case.
Repeat-By:

	Feed the following lines to deroff:
		abc
		.PS < foobar
		def
		ghi

	Deroff should return:
		abc
		def
		ghi

	bug actually returns
		abc

Fix:
	What follows is a contextual diff
*** /usr/src/usr.bin/deroff.c	Mon Dec  9 16:01:47 1985
--- deroff.c	Tue Feb 18 14:34:03 1986
***************
*** 922,927 ****
--- 922,934 ----
  }
  PS()
  {
+ 	for (C ; c == ' ' || c == '\t';C)
+ 		;		/* skip white space */
+ 	if (c == '<'){		/* construct is .PS < file -- don't expect 
+ 				   a .PE */
+ 		SKIP;
+ 		return(0);		   
+ 	}
  	if (!msflag) {
  		inpic();
  	} else {
-- 
Neil Rhodes
UC San Diego

neil@ucsd
{ucbvax,decvax}!sdcsvax!neil