This is the mail archive of the
mauve-discuss@sourceware.org
mailing list for the Mauve project.
Bug in getEncoding testcase
- From: Torben dot Nielsen at sagemdenmark dot dk
- To: mauve-discuss at sourceware dot org
- Date: Tue, 27 Sep 2005 10:20:32 +0200
- Subject: Bug in getEncoding testcase
A few weeks ago I posted this patch to mauve-patches. Since it is still
unchanged in cvs, I would like to ask if I posted to the wrong list, or
if there is a problem with the patch. The problem with the existing
testcase
is that contradictory to the test description in the top of the file, it
requires the extended charsets to be present, causing numerous false error
reports.
Index: gnu/testlet/java/io/InputStreamReader/getEncoding.java
===================================================================
RCS file:
/cvs/mauve/mauve/gnu/testlet/java/io/InputStreamReader/getEncoding.java,v
retrieving revision 1.1
diff -u -F^f -r1.1 getEncoding.java
--- gnu/testlet/java/io/InputStreamReader/getEncoding.java 17 Jun 2005
01:49:48 -0000 1.1
+++ gnu/testlet/java/io/InputStreamReader/getEncoding.java 7 Sep 2005
14:06:06 -0000
@@ -165,8 +165,8 @@
} catch (UnsupportedCharsetException uce) {
supported = false;
}
- harness.check(name, extIoNames[i]);
- harness.check(supported, true);
+ if ( supported )
+ harness.check(name, extIoNames[i]);
}
}