code style

Author: rbri

src/main/java/org/htmlunit/html/impl/SimpleRange.java

@@ -210,14 +210,6 @@ public void collapse(final boolean toStart) {
         }
     }
 
-//    /**
-//     * {@inheritDoc}
-//     */
-//    @Override
-//    public short compareBoundaryPoints(final short how, final Range sourceRange) throws DOMException {
-//        throw new RuntimeException("Not implemented!");
-//    }
-
     /**
      * Removes the contents of this range from the containing document or
      * document fragment without returning a reference to the removed
@@ -281,14 +273,6 @@ else if (endContainer_.getChildNodes().getLength() > endOffset_) {
         }
     }
 
-//    /**
-//     * {@inheritDoc}
-//     */
-//    @Override
-//    public void detach() throws DOMException {
-//        throw new RuntimeException("Not implemented!");
-//    }
-
     /**
      * Moves the contents of a Range from the containing document or document
      * fragment to a new DocumentFragment.
@@ -438,22 +422,6 @@ public void setEnd(final DomNode refNode, final int offset) {
         endContainer_ = refNode;
         endOffset_ = offset;
     }
-//
-//    /**
-//     * {@inheritDoc}
-//     */
-//    @Override
-//    public void setEndAfter(final Node refNode) throws RangeException, DOMException {
-//        throw new RuntimeException("Not implemented!");
-//    }
-//
-//    /**
-//     * {@inheritDoc}
-//     */
-//    @Override
-//    public void setEndBefore(final Node refNode) throws RangeException, DOMException {
-//        throw new RuntimeException("Not implemented!");
-//    }
 
     /**
      * Sets the attributes describing the start.
@@ -465,22 +433,6 @@ public void setStart(final DomNode refNode, final int offset) {
         startOffset_ = offset;
     }
 
-//    /**
-//     * {@inheritDoc}
-//     */
-//    @Override
-//    public void setStartAfter(final Node refNode) throws RangeException, DOMException {
-//        throw new RuntimeException("Not implemented!");
-//    }
-//
-//    /**
-//     * {@inheritDoc}
-//     */
-//    @Override
-//    public void setStartBefore(final Node refNode) throws RangeException, DOMException {
-//        throw new RuntimeException("Not implemented!");
-//    }
-
     /**
      * Reparents the contents of the Range to the given node and inserts the
      * node at the position of the start of the Range.

src/main/java/org/htmlunit/javascript/host/dom/Range.java

@@ -351,8 +351,7 @@ public Object extractContents() {
     }
 
     /**
-     * Returns a W3C {@link org.w3c.dom.ranges.Range} version of this object.
-     * @return a W3C {@link org.w3c.dom.ranges.Range} version of this object
+     * @return a {@link SimpleRange} version of this object
      */
     public SimpleRange getSimpleRange() {
         return new SimpleRange(startContainer_.getDomNodeOrNull(), startOffset_,

src/test/java/org/htmlunit/archunit/ArchitectureTest.java

@@ -247,6 +247,14 @@ public void check(final JavaMethod method, final ConditionEvents events) {
                 .doNotHaveFullyQualifiedName("org.htmlunit.SgmlPage")
             .should().dependOnClassesThat().haveFullyQualifiedName("org.w3c.dom.traversal.DocumentTraversal");
 
+    /**
+     * Make sure to not use javax.imageio.
+     */
+    @ArchTest
+    public static final ArchRule androidRanges = noClasses()
+        .should().dependOnClassesThat().resideInAnyPackage("org.w3c.dom.ranges..");
+
+
     /**
      * Make sure to not use javax.imageio.
      */