public inbox for mauve-discuss@sourceware.org
 help / color / mirror / Atom feed
* Bug in getEncoding testcase
@ 2005-09-27  8:20 Torben.Nielsen
  2005-09-27  8:53 ` David Gilbert
  0 siblings, 1 reply; 2+ messages in thread
From: Torben.Nielsen @ 2005-09-27  8:20 UTC (permalink / raw)
  To: mauve-discuss

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]);
                }
        }


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-09-27  8:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-09-27  8:20 Bug in getEncoding testcase Torben.Nielsen
2005-09-27  8:53 ` David Gilbert

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).