classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[cp-patches] Re: [commit-cp] classpath ./ChangeLog java/util/zip/ZipEntr


From: Mark Wielaard
Subject: [cp-patches] Re: [commit-cp] classpath ./ChangeLog java/util/zip/ZipEntry.ja...
Date: Wed, 31 Aug 2005 17:54:38 +0200

On Wed, 2005-08-31 at 17:44 +0200, Mark Wielaard wrote:
> Hmmm, after a clean build and install of both classpath and jamvm I have
> the same issue. Reverting the patch makes things work again.
> Embarrassing. Apologies. I am investigating what goes wrong and why I
> didn't notice this before.

It was because I had the following patch in my working directory.

Index: java/nio/charset/Charset.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/nio/charset/Charset.java,v
retrieving revision 1.23
diff -u -r1.23 Charset.java
--- java/nio/charset/Charset.java       2 Jul 2005 20:32:40 -0000       1.23
+++ java/nio/charset/Charset.java       31 Aug 2005 15:48:02 -0000
@@ -278,7 +278,7 @@
           }
         catch (Exception e)
           {
-            throw new RuntimeException(e);
+            return new CharsetProvider[] { provider() };
           }
       }
     return providers;

I'll see if I can come up with a good/real workaround.
The issue is that since we are using the UTF-8 encoder now in zip
reading Charset tries to lookup additional providers during bootstrap
which tries to reopen the zip file we are just reading...

Cheers,

Mark

Attachment: signature.asc
Description: This is a digitally signed message part


reply via email to

[Prev in Thread] Current Thread [Next in Thread]