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

henrib pushed a commit to branch JEXL-381
in repository https://gitbox.apache.org/repos/asf/commons-jexl.git

commit dc190a909b465c6838003db227524c3bd5f83e3f
Author: henrib <[email protected]>
AuthorDate: Fri Oct 21 19:33:48 2022 +0200

    JEXL-381: rebased;
---
 .../java/org/apache/commons/jexl3/JexlBuilder.java | 38 +++++++++-------------
 1 file changed, 16 insertions(+), 22 deletions(-)

diff --git a/src/main/java/org/apache/commons/jexl3/JexlBuilder.java 
b/src/main/java/org/apache/commons/jexl3/JexlBuilder.java
index a183d402..00330c59 100644
--- a/src/main/java/org/apache/commons/jexl3/JexlBuilder.java
+++ b/src/main/java/org/apache/commons/jexl3/JexlBuilder.java
@@ -31,31 +31,25 @@ import java.nio.charset.Charset;
 /**
  * Configures and builds a JexlEngine.
  *
- * <p>
- *     The builder allow fine-tuning an engine instance behavior according to 
various control needs.
- *     Check <em>{@link #JexlBuilder()}</em> for permission impacts starting 
with <em>JEXL 3.3</em>.
- * </p><p>
- *     Broad configurations elements are controlled through the features 
({@link JexlFeatures}) that can restrict JEXL
+ * <p>The builder allow fine-tuning an engine instance behavior according to 
various control needs.</p>
+ *
+ * <p>Broad configurations elements are controlled through the features 
({@link JexlFeatures}) that can restrict JEXL
  *  syntax - for instance, only expressions with no-side effects - and 
permissions ({@link JexlPermissions}) that control
- *  the visible set of objects - for instance, avoiding access to any object 
in java.rmi.* -.
- *  </p><p>
- *     Fine error control and runtime-overridable behaviors are implemented 
through options ({@link JexlOptions}). Most
+ *  the visible set of objects - for instance, avoiding access to any object 
in java.rmi.* -. </p>
+ *
+ * <p>Fine error control and runtime-overridable behaviors are implemented 
through options ({@link JexlOptions}). Most
  * common flags accessible from the builder are reflected in its options 
({@link #options()}).
- * </p><p>
- *     The <code>silent</code> flag tells the engine what to do with the 
error; when true, errors are logged as
- * warning, when false, they throw {@link JexlException} exceptions.
- * </p><p>
- *     The <code>strict</code> flag tells the engine when and if null as 
operand is considered an error. The <code>safe</code>
+ * <p>The <code>silent</code> flag tells the engine what to do with the error; 
when true, errors are logged as
+ * warning, when false, they throw {@link JexlException} exceptions.</p>
+ * <p>The <code>strict</code> flag tells the engine when and if null as 
operand is considered an error. The <code>safe</code>
  * flog determines if safe-navigation is used. Safe-navigation allows an  
evaluation shortcut and return null in expressions
- * that attempts dereferencing null, typically a method call or accessing a 
property.
- * </p><p>
- *     The <code>lexical</code> and <code>lexicalShade</code> flags can be 
used to enforce a lexical scope for
+ * that attempts dereferencing null, typically a method call or accessing a 
property.</p>
+ * <p>The <code>lexical</code> and <code>lexicalShade</code> flags can be used 
to enforce a lexical scope for
  * variables and parameters. The <code>lexicalShade</code> can be used to 
further ensure no global variable can be
  * used with the same name as a local one even after it goes out of scope. The 
corresponding feature flags should be
- * preferred since they will detect violations at parsing time. (see {@link 
JexlFeatures})
- * </p><p>
- *     The following rules apply on silent and strict flags:
- * </p>
+ * preferred since they will detect violations at parsing time. (see {@link 
JexlFeatures})</p>
+ *
+ * <p>The following rules apply on silent and strict flags:</p>
  * <ul>
  * <li>When "silent" &amp; "not-strict":
  * <p> 0 &amp; null should be indicators of "default" values so that even in 
an case of error,
@@ -210,7 +204,7 @@ public class JexlBuilder {
     }
 
     /**
-     * Sets the JexlUberspect strategy the engine will use.
+     * Sets the JexlUberspect strategy strategy the engine will use.
      * <p>This is ignored if the uberspect has been set.
      *
      * @param rs the strategy
@@ -221,7 +215,7 @@ public class JexlBuilder {
         return this;
     }
 
-    /** @return the JexlUberspect strategy */
+    /** @return the strategy strategy */
     public JexlUberspect.ResolverStrategy strategy() {
         return this.strategy;
     }

Reply via email to