From d87bed0e560a8b54eb54500eb8cff681b09f4b2f Mon Sep 17 00:00:00 2001 From: "opencode-agent[bot]" Date: Sun, 17 May 2026 15:20:46 +0000 Subject: [PATCH] Fixed 2 bugs in XMLElement.java - Added missing break statement in scanWhitespace (line 2304) to prevent fallthrough from \n case to \r case - Replaced swallowed IOException with AssertionError in parseString methods (lines 1716, 1824) - fails fast if assumptions change --- core/src/endorsed/nanoxml/org/jnode/nanoxml/XMLElement.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/core/src/endorsed/nanoxml/org/jnode/nanoxml/XMLElement.java b/core/src/endorsed/nanoxml/org/jnode/nanoxml/XMLElement.java index de83ba3f12..e1ba408ce5 100644 --- a/core/src/endorsed/nanoxml/org/jnode/nanoxml/XMLElement.java +++ b/core/src/endorsed/nanoxml/org/jnode/nanoxml/XMLElement.java @@ -1713,7 +1713,7 @@ public void parseString(String string) this.parseFromReader(new StringReader(string), /*startingLineNr*/ 1); } catch (IOException e) { - // Java exception handling suxx + throw new AssertionError(e); } } @@ -1821,7 +1821,7 @@ public void parseString(String string, try { this.parseFromReader(new StringReader(string), startingLineNr); } catch (IOException e) { - // Java exception handling suxx + throw new AssertionError(e); } } @@ -2301,6 +2301,7 @@ protected char scanWhitespace(StringBuffer result) case '\t': case '\n': result.append(ch); + break; case '\r': break; default: