Path: utzoo!utgpu!attcan!uunet!portal!atari!apratt
From: apratt@atari.UUCP (Allan Pratt)
Newsgroups: comp.sources.games.bugs
Subject: Sokoban restart bug & fix
Message-ID: <1295@atari.UUCP>
Date: 6 Jan 89 01:59:13 GMT
Organization: Atari Corp., Sunnyvale CA
Lines: 20

I didn't need the patch which appeared here for Sokoban relating to
cbreak mode; I don't know why, but my version works fine (on 4.2BSD). 
However... 

Sokoban is supposed to start you at the level after the last one you
successfully completed, or another one you specify on the command line,
but no higher than the default.  The procedure which determines the
level you've achieved has a bug, though: it uses a variable before
setting it. 

Here's the fix, to file score.c:

136c136
<    for( i = 0; (i < scoreentries) && (! found); i++)
---
>    for( found = i = 0; (i < scoreentries) && (! found); i++)

============================================
Opinions expressed above do not necessarily	-- Allan Pratt, Atari Corp.
reflect those of Atari Corp. or anyone else.	  ...ames!atari!apratt