Skip to content

Commit 13534a3

Browse files
committed
fix some pmd warnings
1 parent a31f2a9 commit 13534a3

File tree

3 files changed

+14
-12
lines changed

3 files changed

+14
-12
lines changed

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

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,18 @@ public class DOMException extends HtmlUnitScriptable {
177177
* Default constructor used to build the prototype.
178178
*/
179179
public DOMException() {
180+
super();
181+
}
182+
183+
/**
184+
* Constructor.
185+
* @param message the exception message
186+
* @param error the error code (on of the constants from the class)
187+
*/
188+
public DOMException(final String message, final int error) {
189+
message_ = message;
190+
code_ = error;
191+
name_ = COMMON_ERROR_NAMES.get(error - 1);
180192
}
181193

182194
/**
@@ -214,17 +226,6 @@ public void jsConstructor(final String message, final Object error) {
214226
code_ = 0;
215227
}
216228

217-
/**
218-
* Constructor.
219-
* @param message the exception message
220-
* @param error the error code (on of the constants from the class)
221-
*/
222-
public DOMException(final String message, final int error) {
223-
message_ = message;
224-
code_ = error;
225-
name_ = COMMON_ERROR_NAMES.get(error - 1);
226-
}
227-
228229
/**
229230
* Gets the exception code.
230231
* @return the exception code

src/main/java/org/htmlunit/javascript/host/html/HTMLAllCollection.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ public HTMLAllCollection() {
5353
*/
5454
@JsxConstructor
5555
public void jsConstructor() {
56+
// nothing to do
5657
}
5758

5859
/**

src/main/java/org/htmlunit/util/OrderedFastHashMap.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ else if (k.equals(key)) {
200200
final Object ret = mapData_[ptr + 1];
201201
mapData_[ptr + 1] = value;
202202

203-
/// existing entry, no need to update the position
203+
// existing entry, no need to update the position
204204

205205
return (V) ret;
206206
}

0 commit comments

Comments
 (0)