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

> 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.

- Raw text -