gnupic: Re: [gnupic] Fwd: Patches for gputils-0.13.4
Subject:
Re: [gnupic] Fwd: Patches for gputils-0.13.4
From:
David ####@####.####
Date:
14 Nov 2007 03:24:46 +0000
Message-Id: <20071113222330.1286fc16@DEEPTHOUGHT.BARNET.net>
On Wed, 11 Jul 2007 10:55:54 -0700
Andreas Kabel ####@####.#### wrote:
> >>> ---------------------------- Original Message
> >>> ----------------------------
> >>> Subject: Patches for gputils-0.13.4
> >>> From: "Andreas Kabel" ####@####.####
> >>> Date: Tue, July 10, 2007 10:59 pm
> >>> To: ####@####.####
> >>> --------------------------------------------------------------------------
> >>> Anyway, I have a set of patches for gplink/gpasm which allow for
> >>> support of Microchip's new Coff format; perhaps this is something
> >>> which should go in any new release of the gputils.
Andreas, I've finally gotten around to merging your changes into the
gputils SVN repository. They're included in rev 510. A few more details
are attached in the patch tracker:
https://sourceforge.net/support/tracker.php?aid=1752107
My fault for missing the 0.13.5 release, but I plan to release 0.14.0
soon with all of these new features.
@all: Please test extensively ('gpasm -C' generates new-style COFF).
> I recently noticed, however, that all COFF-related stuff is
> platform-dependent, as it assumes that sizeof(long int)==4 and
> sizeof(short)==2, which is not necessarily true. It might make sense
> to rework the patch to take care of this issue.
I didn't touch this issue, but it probably makes sense to treat it
separately. For one thing, it would help if whoever made the changes
had a different platform that had size-related failures so that we
could do at least a little testing. Also, I believe the size changes
affect more than just the lines changed in the patch.
If you'd like to revisit the issue yourself, I promise to get it into
SVN right away this time =).
David Barnett