From 6dfb90cf55f7a7abfcbe6a4bf80b38569cc2a8b6 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 6 Jul 2005 20:10:41 +0000 Subject: [PATCH] InputStreamReader.java (refill): Handle no-progress case correctly. * java/io/InputStreamReader.java (refill): Handle no-progress case correctly. * gnu/gcj/convert/IOConverter.java: Add 'utf8' alias. From-SVN: r101663 --- libjava/ChangeLog | 6 ++++++ libjava/gnu/gcj/convert/IOConverter.java | 4 +++- libjava/java/io/InputStreamReader.java | 20 +++++++++++++++++--- 3 files changed, 26 insertions(+), 4 deletions(-) diff --git a/libjava/ChangeLog b/libjava/ChangeLog index 39e2fd0736e..657e238d879 100644 --- a/libjava/ChangeLog +++ b/libjava/ChangeLog @@ -1,3 +1,9 @@ +2005-07-06 Tom Tromey + + * java/io/InputStreamReader.java (refill): Handle no-progress + case correctly. + * gnu/gcj/convert/IOConverter.java: Add 'utf8' alias. + 2005-07-06 Tom Tromey * testsuite/libjava.jacks/jacks.xfail: Removed 9.1.3-body-5. diff --git a/libjava/gnu/gcj/convert/IOConverter.java b/libjava/gnu/gcj/convert/IOConverter.java index ba3260ac17e..2c6849ce7a3 100644 --- a/libjava/gnu/gcj/convert/IOConverter.java +++ b/libjava/gnu/gcj/convert/IOConverter.java @@ -1,4 +1,4 @@ -/* Copyright (C) 2000, 2001 Free Software Foundation +/* Copyright (C) 2000, 2001, 2005 Free Software Foundation This file is part of libgcj. @@ -28,6 +28,8 @@ public abstract class IOConverter // canonical name. hash.put ("iso-latin-1", "8859_1"); hash.put ("iso8859_1", "8859_1"); + // At least one build script out there uses 'utf8'. + hash.put ("utf8", "UTF8"); // On Solaris the default encoding, as returned by nl_langinfo(), // is `646' (aka ASCII), but the Solaris iconv_open() doesn't // understand that. We work around the problem by adding an diff --git a/libjava/java/io/InputStreamReader.java b/libjava/java/io/InputStreamReader.java index f8ad53cdd01..e55f1352caa 100644 --- a/libjava/java/io/InputStreamReader.java +++ b/libjava/java/io/InputStreamReader.java @@ -289,9 +289,23 @@ public class InputStreamReader extends Reader return -1; converter.setInput(in.buf, in.pos, in.count); int count = converter.read(buf, offset, length); - in.skip(converter.inpos - in.pos); - if (count > 0) - return count; + + // We might have bytes but not have made any progress. In + // this case we try to refill. If refilling fails, we assume + // we have a malformed character at the end of the stream. + if (count == 0 && converter.inpos == in.pos) + { + in.mark(in.count); + if (! in.refill ()) + throw new CharConversionException (); + in.reset(); + } + else + { + in.skip(converter.inpos - in.pos); + if (count > 0) + return count; + } } } } -- 2.30.2