You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2021/04/20 17:33:48 UTC
[tomcat] 04/04: Fix some optimisations around omit.
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit e6353864e6245da2553e6f56c07819dd8e703607
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue Apr 20 16:45:52 2021 +0100
Fix some optimisations around omit.
The optimisations were not applied because the check for the string
literals "true" and "false" did not account for the tested values being
quoted.
---
java/org/apache/jasper/compiler/Generator.java | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/java/org/apache/jasper/compiler/Generator.java b/java/org/apache/jasper/compiler/Generator.java
index 093cde3..3fdf3c4 100644
--- a/java/org/apache/jasper/compiler/Generator.java
+++ b/java/org/apache/jasper/compiler/Generator.java
@@ -2016,14 +2016,16 @@ class Generator {
if (omitAttr == null) {
omit = "false";
} else {
+ // String literals returned by attributeValue will be
+ // quoted and escaped.
omit = attributeValue(omitAttr, false, boolean.class);
- if ("true".equals(omit)) {
+ if ("\"true\"".equals(omit)) {
continue;
}
}
value = generateNamedAttributeValue(
attrs[i].getNamedAttributeNode());
- if ("false".equals(omit)) {
+ if ("\"false\"".equals(omit)) {
nvp = " + \" " + attrs[i].getName() + "=\\\"\" + " +
value + " + \"\\\"\"";
} else {
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org