Yes, the HelloWorld.pack is different and needs to be replaced with
the one in the bug report.
Thanks,
Alex.
On 13/09/06, Mikhail Loenko (JIRA) <[EMAIL PROTECTED]> wrote:
[ http://issues.apache.org/jira/browse/HARMONY-1386?page=all ]
Mikhail Loenko resolved HARMONY-1386.
-------------------------------------
Resolution: Fixed
Thanks for the patch!
Ive applied it in revision 442918
the tests passed
it's not clear what to do with HelloWorld.pack. should I replace that one in
SVN?
> Update to Pack200 decoding libraries
> ------------------------------------
>
> Key: HARMONY-1386
> URL: http://issues.apache.org/jira/browse/HARMONY-1386
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Reporter: Alex Blewitt
> Assigned To: Mikhail Loenko
> Priority: Minor
> Attachments: HelloWorld.pack, HelloWorld.pack, pack200.patch,
pack200.patch
>
>
> I've updated the Pack200 decoding to take into account of the dynamic codec
switching (although it currently throws an error if it's detected that such a
dynamic switch is used) as well as adding more descriptive logging of what bands
are being parsed, including remaining unimplemented bands. No output should be
seen unless -Ddebug.pack200=true, but this is only for use whilst ensuring the
unpacking is right.
> I'm not sure whether the patch contains the updated binary for
HelloWorld.pack, so I'll attach that in addition to the patch.
> I've also fixed a couple of minor bugs in parsing the class and field bands.
> This patch passes all the tests with against the HEAD, but it wouldn't
surprise me if the patch was missing some required file :-(
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
Terms of use : http://incubator.apache.org/harmony/mailing.html
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
