Xref: utzoo comp.sources.bugs:696 rec.games.empire:302
Path: utzoo!mnetor!uunet!husc6!rutgers!clyde!cbosgd!osu-cis!tut!perch!stucki
From: stucki@perch.cis.ohio-state.edu (David J. Stucki)
Newsgroups: comp.sources.bugs,rec.games.empire
Subject: latest conquest bug report
Message-ID: <5030@tut.cis.ohio-state.edu>
Date: 21 Jan 88 20:32:38 GMT
Sender: news@tut.cis.ohio-state.edu
Organization: The Ohio State University Dept of Computer and Information Science
Lines: 25



Here is the latest bug in (Barlow's) conquest I found that I haven't
seen reported elsewhere.

It doesn't show up as long as the world is square, i.e., 48x48 or 64x64, etc.
However, we tried to set it up as 64x32 and it came back with floating
exception in main.c at a call to refresh();

Well, it appears that there are several places (10 to 15) where a reference
to MAPX is made when it should be MAPY. Something like this:

	for(i=0;i<MAPX;i++) for(j=0;j<MAPX;j++) fought[i][j]=0;
where it should be 
	for(i=0;i<MAPX;i++) for(j=0;j<MAPY;j++) fought[i][j]=0;
(this one was in combat.c)

We went through and found them all: do a fgrep MAXY on files combat.c
move.c reports.c and update.c. That was all the ones as far as we can tell
because now it works again.
-=-
~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~+~
David J Stucki					c/o Dept. Computer and 
P.O. Box 713 Park Hall				    Information Science
110 W. 11th Ave.				    2036 Neil Ave.