From 63625b3c622407273d2654660be7659ac5d74db7 Mon Sep 17 00:00:00 2001
From: Simulant <nfaupel.dev@gmail.com>
Date: Tue, 5 Mar 2024 09:43:54 +0100
Subject: [PATCH] #863 improve performance of JSONTokener#nextString

replacing a switch-case statement with few branches
by if-else cases
---
 src/main/java/org/json/JSONTokener.java | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/src/main/java/org/json/JSONTokener.java b/src/main/java/org/json/JSONTokener.java
index 0bc6dfb..96dc71c 100644
--- a/src/main/java/org/json/JSONTokener.java
+++ b/src/main/java/org/json/JSONTokener.java
@@ -295,12 +295,9 @@ public class JSONTokener {
         StringBuilder sb = new StringBuilder();
         for (;;) {
             c = this.next();
-            switch (c) {
-            case 0:
-            case '\n':
-            case '\r':
-                throw this.syntaxError("Unterminated string");
-            case '\\':
+            if (c == quote) {
+                return sb.toString();
+            } else if (c == '\\') {
                 c = this.next();
                 switch (c) {
                 case 'b':
@@ -334,11 +331,9 @@ public class JSONTokener {
                 default:
                     throw this.syntaxError("Illegal escape.");
                 }
-                break;
-            default:
-                if (c == quote) {
-                    return sb.toString();
-                }
+            } else if (c == 0 || c == '\n' || c == '\r') {
+                throw this.syntaxError("Unterminated string");
+            } else {
                 sb.append(c);
             }
         }