Skip to content

Commit a828453

Browse files
committed
findbugs warning cleanup
1 parent a695b5f commit a828453

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/main/java/org/owasp/html/HtmlElementTables.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -532,6 +532,7 @@ static boolean binSearchRange(
532532
int lo = leftIncl, hi = rightExcl;
533533
while (lo < hi) {
534534
int mid = (lo + hi) >> 1;
535+
assert mid >= lo;
535536
int el = arr[mid];
536537
int delta = value - el;
537538
if (delta == 0) {

src/test/java/org/owasp/html/CssFuzzerTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,9 +64,8 @@ final class Watcher implements Runnable {
6464
public void run() {
6565
synchronized (this) {
6666
try {
67-
while (true) {
67+
while (input == null) {
6868
this.wait(1000 /* ms = 1s */);
69-
if (input == null) { break; } // Done
7069
long now = System.nanoTime();
7170
if (now - started >= 1000000000L /* ns = 1s */) {
7271
System.err.println(

0 commit comments

Comments
 (0)