commit 9235fbce7c604052eace4c3ed083b9965a0c1f86
author Josh Tynjala <joshtynjala@apache.org> Thu Dec 17 14:48:39 2020 -0800
committer Josh Tynjala <joshtynjala@apache.org> Thu Dec 17 14:48:39 2020 -0800
tree ef8b883eb4134efec8f7f632e379004f8284b405
parent b30228b75697d670e265d370037f591e6e16c14d

ClosureUtils: fix bindable variables not being detected as accessors

compiler-jx/src/main/java/org/apache/royale/compiler/utils/ClosureUtils.java

diff --git a/compiler-jx/src/main/java/org/apache/royale/compiler/utils/ClosureUtils.java b/compiler-jx/src/main/java/org/apache/royale/compiler/utils/ClosureUtils.java
index 1ae51eb..fdf611c 100644
--- a/compiler-jx/src/main/java/org/apache/royale/compiler/utils/ClosureUtils.java
+++ b/compiler-jx/src/main/java/org/apache/royale/compiler/utils/ClosureUtils.java
@@ -115,8 +115,10 @@
                             boolean isMethod = localDef instanceof IFunctionDefinition
                                 && !(localDef instanceof IAccessorDefinition);
                             boolean isVar = localDef instanceof IVariableDefinition
-                                && !(localDef instanceof IAccessorDefinition);
-                            boolean isAccessor = localDef instanceof IAccessorDefinition;
+                                && !(localDef instanceof IAccessorDefinition)
+                                && !localDef.isBindable();
+                            boolean isAccessor = localDef instanceof IAccessorDefinition
+                                || (localDef instanceof IVariableDefinition && localDef.isBindable());
                             if(localDef.isPublic() || isCustomNS)
                             {
                                 if(!preventRenamePublicSymbols)
@@ -317,8 +319,10 @@
                             boolean isMethod = localDef instanceof IFunctionDefinition
                                 && !(localDef instanceof IAccessorDefinition);
                             boolean isVar = localDef instanceof IVariableDefinition
-                                && !(localDef instanceof IAccessorDefinition);
-                            boolean isAccessor = localDef instanceof IAccessorDefinition;
+                                && !(localDef instanceof IAccessorDefinition)
+                                && !localDef.isBindable();
+                            boolean isAccessor = localDef instanceof IAccessorDefinition
+                                || (localDef instanceof IVariableDefinition && localDef.isBindable());
                             if (isMethod || isVar || isAccessor)
                             {
                                 INamespaceReference nsRef = localDef.getNamespaceReference();