You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by he...@apache.org on 2018/09/10 14:36:46 UTC

[commons-jexl] branch master updated (2fdb427 -> 8603a6d)

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

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


    from 2fdb427  JEXL-271: simplified curry(...) after being hinted it was a form of closure
     new e51bcaa  JEXL-271, JEXL-270: release notes, changes
     new 8603a6d  JEXL: enable easier log lib change and package rename

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 RELEASE-NOTES.txt                                        |  6 ++++--
 src/site/xdoc/changes.xml                                |  6 ++++++
 src/test/java/org/apache/commons/jexl3/CaptureLog.java   | 13 +++++++++++++
 src/test/java/org/apache/commons/jexl3/ClassCreator.java |  2 +-
 4 files changed, 24 insertions(+), 3 deletions(-)


[commons-jexl] 01/02: JEXL-271, JEXL-270: release notes, changes

Posted by he...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit e51bcaa6084e92fbb3ad98e46bbc4d97a03264d9
Author: henrib <he...@apache.org>
AuthorDate: Mon Sep 10 16:02:54 2018 +0200

    JEXL-271, JEXL-270: release notes, changes
---
 RELEASE-NOTES.txt         | 6 ++++--
 src/site/xdoc/changes.xml | 6 ++++++
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/RELEASE-NOTES.txt b/RELEASE-NOTES.txt
index 08d54ed..85db7c9 100644
--- a/RELEASE-NOTES.txt
+++ b/RELEASE-NOTES.txt
@@ -42,20 +42,22 @@ New Features in 3.2:
 * JEXL-264:      Allow space, quote & double-quote in identifiers
 * JEXL-260:      Automatically inject JexlContext in constructor call when possible
 * JEXL-252:      Allow for interpolated strings to be used in property access operators
-* JEXL-250:      Safe navigation operator
+* JEXL-250:      Safe navigation operator (?.)
 * JEXL-248:      Allow range subexpression as an array property assignment identifier
 * JEXL-243:      Allow restricting available features in Script/Expressions
 * JEXL-238:      Restrict getLiteralClass to a Number for NumberLiterals
 * JEXL-237:      Ability to restrict usage of certain names when declaring local variables
 * JEXL-236:      Support CharSequence in size(), empty() and contains() operators
 * JEXL-234:      Extend application of operators startsWith and endsWith from String to CharSequence types
-* JEXL-226:      add ?? operator support
+* JEXL-226:      Add ?? operator support
 * JEXL-224:      The ability to overload call() operator in customized JexlArithmetic implementation
 * JEXL-212:      Restrict usage of assignment statements in JexlExpression
 
 Bugs Fixed in 3.2:
 ==================
 
+* JEXL-271:      Hoisted variable is lost when currying lambda
+* JEXL-270:      Wrong Script$Curried creation when script.curry() method is called inside script
 * JEXL-261:      JexlEngine.setClassLoader(...) should reload namespaces that are classes
 * JEXL-246:      Intermittent ambiguous method invocation when processing assignOverload
 * JEXL-245:      Engine in strict mode fails to fail on unsolvable variables or properties
diff --git a/src/site/xdoc/changes.xml b/src/site/xdoc/changes.xml
index 6ea1dfe..c320979 100644
--- a/src/site/xdoc/changes.xml
+++ b/src/site/xdoc/changes.xml
@@ -26,6 +26,12 @@
     </properties>
     <body>
         <release version="3.2" date="unreleased">
+            <action dev="henrib" type="fix" issue="JEXL-271" due-to="Dmitri Blinov">
+                Hoisted variable is lost when currying lambda
+            </action>
+            <action dev="henrib" type="fix" issue="JEXL-270" due-to="Dmitri Blinov">
+            Wrong Script$Curried creation when script.curry() method is called inside script
+            </action>
             <action dev="henrib" type="add" issue="JEXL-264">
                 Allow space, quote and double-quote in identifiers
             </action>


[commons-jexl] 02/02: JEXL: enable easier log lib change and package rename

Posted by he...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

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

commit 8603a6d45ba5573c4becfdf5a0780dff38bd72aa
Author: henrib <he...@apache.org>
AuthorDate: Mon Sep 10 16:36:22 2018 +0200

    JEXL: enable easier log lib change and package rename
---
 src/test/java/org/apache/commons/jexl3/CaptureLog.java   | 13 +++++++++++++
 src/test/java/org/apache/commons/jexl3/ClassCreator.java |  2 +-
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/src/test/java/org/apache/commons/jexl3/CaptureLog.java b/src/test/java/org/apache/commons/jexl3/CaptureLog.java
index 2ae0aa6..b216c65 100644
--- a/src/test/java/org/apache/commons/jexl3/CaptureLog.java
+++ b/src/test/java/org/apache/commons/jexl3/CaptureLog.java
@@ -29,6 +29,14 @@ public class CaptureLog implements Log {
         StackTraceElement[] stack = new Exception().fillInStackTrace().getStackTrace();
         return stack[2];
     }
+   
+    public CaptureLog() {
+        this("org.apache.commons.jexl3");
+    }
+    
+    public CaptureLog(String name) {
+        //super(name);
+    }
 
     public boolean isEmpty() {
         return captured.isEmpty();
@@ -43,6 +51,11 @@ public class CaptureLog implements Log {
         }
         return count;
     }
+        
+    //@Override
+    public boolean isEnabledFor(int /*Priority*/ p) {
+        return true;
+    }
 
     @Override
     public void debug(Object o) {
diff --git a/src/test/java/org/apache/commons/jexl3/ClassCreator.java b/src/test/java/org/apache/commons/jexl3/ClassCreator.java
index 4303ed0..1a3bf2f 100644
--- a/src/test/java/org/apache/commons/jexl3/ClassCreator.java
+++ b/src/test/java/org/apache/commons/jexl3/ClassCreator.java
@@ -49,8 +49,8 @@ public class ClassCreator {
     public static final boolean canRun = true;//comSunToolsJavacMain();
 
     static final String JEXL_PACKAGE = "org.apache.commons.jexl3";
-    static final String GEN_PATH = "/org/apache/commons/jexl3/generated";
     static final String GEN_PACKAGE = "org.apache.commons.jexl3.generated";
+    static final String GEN_PATH = "/" + GEN_PACKAGE.replace(".", "/");///org/apache/commons/jexl3/generated";
     static final String GEN_CLASS = GEN_PACKAGE + ".";
 
     /**