X-Authentication-Warning: kendall.sfbr.org: jeffw set sender to jeffw AT darwin DOT sfbr DOT org using -f
Date: Fri, 26 Jan 2001 13:28:45 -0600
From: JT Williams <jeffw AT darwin DOT sfbr DOT org>
To: djgpp-workers AT delorie DOT com
Cc: Bill Currie <bill AT taniwha DOT org>, Eli Zaretskii <eliz AT is DOT elta DOT co DOT il>
Subject: Re: djasm patch
Message-ID: <20010126132845.B23418@kendall.sfbr.org>
Mail-Followup-To: djgpp-workers AT delorie DOT com, Bill Currie <bill AT taniwha DOT org>,
	Eli Zaretskii <eliz AT is DOT elta DOT co DOT il>
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> <6137-Fri26Jan2001202056+0200-eliz AT is DOT elta DOT co DOT il>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
User-Agent: Mutt/1.2.5i
In-Reply-To: <6137-Fri26Jan2001202056+0200-eliz@is.elta.co.il>; from eliz@is.elta.co.il on Fri, Jan 26, 2001 at 08:20:57PM +0200
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

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

Aha.  This explains why Bill cannot recall adding it....

I have confirmed that the `if (i<bsspc-1)' clause in src/stub/djasm.y
(marked for deletion in above patch) does *not* appear in any of Bill's
versions of djasm.y.  Since Eli introduced it to fix a bug, we can safely
ignore this patch.

Please bear with me; I'm working my way through Bill's changes one
piece at a time.  I'll try to make the next hunk more exciting.