Path: utzoo!utgpu!water!watmath!clyde!att!osu-cis!tut.cis.ohio-state.edu!mailrus!ames!haven!mimsy!chris
From: chris@mimsy.UUCP (Chris Torek)
Newsgroups: comp.bugs.4bsd
Subject: Re: strange effects with enumerated types
Keywords: enum,courier
Message-ID: <14072@mimsy.UUCP>
Date: 20 Oct 88 00:45:49 GMT
References: <142@csaran.cs.tcd.ie> <20@auspex.UUCP> <14065@mimsy.UUCP>
Organization: U of Maryland, Dept. of Computer Science, Coll. Pk., MD 20742
Lines: 24

In article <14065@mimsy.UUCP> I noted that the bug was in the following
few lines from src/lib/mip/pftn.c:
>	if( scl == class ){
>		if( p->offset!= strucoff++ ) break;
>		psave( idp );
>		}
>	break;

but then added
>...  I *think* that it should return after calling psave(), except that
>I am not sure that, if p->offset != strucoff++, strucoff should have
>been incremented.

Of course not!  The code should read

	if( scl == class && p->offset == strucoff ){
		strucoff++;
		psave( idp );
		return;
		}
	break;
-- 
In-Real-Life: Chris Torek, Univ of MD Comp Sci Dept (+1 301 454 7163)
Domain:	chris@mimsy.umd.edu	Path:	uunet!mimsy!chris