[httpcomponents-client] branch master updated: Avoid redundant validation of '\"'. Its check it before.

[ol...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

olegk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/httpcomponents-client.git


The following commit(s) were added to refs/heads/master by this push:
     new 4c464b2  Avoid redundant validation of '\"'. Its check it before.
4c464b2 is described below

commit 4c464b2432525102ad293c20e636970176b74c28
Author: Arturo Bernal <ar...@gmail.com>
AuthorDate: Sat Dec 11 17:28:14 2021 +0100

    Avoid redundant validation of '\"'. Its check it before.
---
 .../main/java/org/apache/hc/client5/http/impl/cache/WarningValue.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/WarningValue.java b/httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/WarningValue.java
index 6969e6f..c8efd77 100644
--- a/httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/WarningValue.java
+++ b/httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/WarningValue.java
@@ -221,7 +221,7 @@ class WarningValue {
             } else if (c == '\"') {
                 foundEnd = true;
                 offs++;
-            } else if (c != '\"' && !isControl(c)) {
+            } else if (!isControl(c)) {
                 offs++;
             } else {
                 parseError();