This is the mail archive of the mauve-discuss@sourceware.org mailing list for the Mauve project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

childNodesLength patch update


Hi,

Pedro noticed that an old boolean variable was still used from an older
revision of the test. This is not needed anymore. So here is the patch
to remove it:

2006-01-17  Pedro Izecksohn  <izecksohn@yahoo.com>

    * gnu/testlet/org/w3c/dom/childNodesLength.java (passed): Removed.
    (checkNode): Don't check passed variable.

Committed,

Mark
Index: gnu/testlet/org/w3c/dom/childNodesLength.java
===================================================================
RCS file: /cvs/mauve/mauve/gnu/testlet/org/w3c/dom/childNodesLength.java,v
retrieving revision 1.1
diff -u -r1.1 childNodesLength.java
--- gnu/testlet/org/w3c/dom/childNodesLength.java	16 Jan 2006 15:15:45 -0000	1.1
+++ gnu/testlet/org/w3c/dom/childNodesLength.java	17 Jan 2006 08:06:24 -0000
@@ -34,7 +34,6 @@
 {
 
   TestHarness harness;
-  boolean passed;
 
   private void checkNode (Node node)
   {
@@ -85,8 +84,6 @@
 
     this.harness=harness;
 
-    passed=true;
-
     DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
 
     DocumentBuilder db = null;
@@ -129,15 +126,6 @@
 
     recurse (document.getChildNodes());
 
-    if (passed)
-    {
-      harness.check (true, "All Node kinds tested implement .getChildNodes().getLength() correctly.");
-    }
-    else
-    {
-      harness.fail ("Some Node kind does NOT implement .getChildNodes().getLength() correctly.");
-    }
-
   }
 
 }

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]