Date: Fri, 26 Jan 2001 20:20:57 +0200
From: "Eli Zaretskii" <eliz AT is DOT elta DOT co DOT il>
Sender: halo1 AT zahav DOT net DOT il
To: JT Williams <jeffw AT darwin DOT sfbr DOT org>
Message-Id: <6137-Fri26Jan2001202056+0200-eliz@is.elta.co.il>
X-Mailer: Emacs 20.6 (via feedmail 8.3.emacs20_6 I) and Blat ver 1.8.6
CC: djgpp-workers AT delorie DOT com, bill AT taniwha DOT org
In-reply-to: <20010126082741.A22475@kendall.sfbr.org> (message from JT
	Williams on Fri, 26 Jan 2001 08:27:41 -0600)
Subject: Re: djasm patch
References: <20010125161754 DOT A22046 AT kendall DOT sfbr DOT org> <200101260746 DOT IAA04919 AT lws256 DOT lu DOT erisoft DOT se> <20010126082741 DOT A22475 AT kendall DOT sfbr DOT org>
Reply-To: djgpp-workers AT delorie DOT com
Errors-To: nobody AT delorie DOT com
X-Mailing-List: djgpp-workers AT delorie DOT com
X-Unsubscribes-To: listserv AT delorie DOT com
Precedence: bulk

> Date: Fri, 26 Jan 2001 08:27:41 -0600
> From: JT Williams <jeffw AT darwin DOT sfbr DOT org>
> 
> Here's the next small hunk: I believe it *also* represents a
> `backwards in time' patch and should be discarded.  Comments?
> 
> --- djasm.y	Thu Jan 25 16:11:57 2001
> +++ /home/jeffw/tmp/bcurrie/djasm/djasm.y	Fri Jan 26 07:53:09 2001
> @@ -1509,8 +1596,7 @@
>          if ((i&15) == 15)
>          {
>            fputc('\n', outfile);
> -          if (i<bsspc-1)
> -            fputs(leader, outfile);
> +          fputs(leader, outfile);
>          }
>          else
>            if (i<bsspc-1)

Sorry, I'm not sure I understand what do you mean by ``backwards in
time'' here.

If you meant that this patch should be applied with -R, then it's not
necessary: the if clause is already in the current sources (has been
there since May 1999).  If you mean that the if clause needs to be
removed, then I disagree: it was added to djasm.y (by yours truly) to
fix a bug.