This is the mail archive of the
mauve-patches@sourceware.org
mailing list for the Mauve project.
FYI: Little cleanup
- From: Mark Wielaard <mark at klomp dot org>
- To: mauve-patches at sources dot redhat dot com
- Date: Tue, 25 Jul 2006 00:23:29 +0200
- Subject: FYI: Little cleanup
Hi,
Here are some small cleanups that helped me when going through the mauve
results recently:
2006-07-23 Mark Wielaard <mark@klomp.org>
* gnu/testlet/javax/swing/JSpinner/constructors.java:
Remove println.
* gnu/testlet/javax/swing/text/DefaultEditorKit/getActions.java:
Likewise.
* gnu/testlet/javax/swing/text/AbstractDocument/filterTest.java:
Add extra debug output on exception.
Committed,
Mark
Index: gnu/testlet/javax/swing/JSpinner/constructors.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/javax/swing/JSpinner/constructors.java,v
retrieving revision 1.1
diff -u -r1.1 constructors.java
--- gnu/testlet/javax/swing/JSpinner/constructors.java 15 Feb 2006 14:38:32 -0000 1.1
+++ gnu/testlet/javax/swing/JSpinner/constructors.java 24 Jul 2006 21:48:12 -0000
@@ -77,7 +77,6 @@
// some private class is listening and passing on events
EventListener[] mListeners = m.getListeners(ChangeListener.class);
harness.check(mListeners.length, 1);
- System.out.println(mListeners[0]);
harness.check(!Arrays.asList(mListeners).contains(spinner));
harness.check(!Arrays.asList(mListeners).contains(spinner.getUI()));
harness.check(!Arrays.asList(mListeners).contains(spinner.getEditor()));
Index: gnu/testlet/javax/swing/text/DefaultEditorKit/getActions.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/javax/swing/text/DefaultEditorKit/getActions.java,v
retrieving revision 1.2
diff -u -r1.2 getActions.java
--- gnu/testlet/javax/swing/text/DefaultEditorKit/getActions.java 15 Jun 2006 18:02:07 -0000 1.2
+++ gnu/testlet/javax/swing/text/DefaultEditorKit/getActions.java 24 Jul 2006 21:48:12 -0000
@@ -1,6 +1,6 @@
// Tags: JDK1.4
-// Copyright (C) 2004 Michael Koch <konqueror@gmx.de>
+// Copyright (C) 2004, 2006 Michael Koch <konqueror@gmx.de>
// This file is part of Mauve.
@@ -41,8 +41,7 @@
found = true;
break;
}
- System.out.println(found);
- System.out.println(name);
+
if (found) {
h.check(found, name);
}
Index: gnu/testlet/javax/swing/text/AbstractDocument/filterTest.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/javax/swing/text/AbstractDocument/filterTest.java,v
retrieving revision 1.1
diff -u -r1.1 filterTest.java
--- gnu/testlet/javax/swing/text/AbstractDocument/filterTest.java 9 May 2006 20:34:23 -0000 1.1
+++ gnu/testlet/javax/swing/text/AbstractDocument/filterTest.java 24 Jul 2006 21:48:12 -0000
@@ -103,6 +103,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -121,6 +122,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -139,6 +141,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -158,6 +161,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -175,6 +179,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -195,6 +200,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -236,6 +242,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -257,6 +264,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -277,6 +285,7 @@
catch (BadLocationException ble)
{
testException = ble;
+ harness.debug(ble);
}
harness.check(testException, null);
@@ -298,6 +307,7 @@
}
catch(BadLocationException ble)
{
+ harness.debug(ble);
harness.fail("replace6");
}