[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[elpa] externals/phps-mode cefab36fa8 054/212: More work on bookkeeping
|
From: |
Christian Johansson |
|
Subject: |
[elpa] externals/phps-mode cefab36fa8 054/212: More work on bookkeeping arrow function variables |
|
Date: |
Wed, 26 Jan 2022 01:50:42 -0500 (EST) |
branch: externals/phps-mode
commit cefab36fa8925b00a83442d001e079c3f59a54c1
Author: Christian Johansson <christian@cvj.se>
Commit: Christian Johansson <christian@cvj.se>
More work on bookkeeping arrow function variables
---
phps-mode-ast-bookkeeping.el | 335 ++++++++++++++++++++++++-------------------
test/phps-mode-test-ast.el | 3 +
2 files changed, 194 insertions(+), 144 deletions(-)
diff --git a/phps-mode-ast-bookkeeping.el b/phps-mode-ast-bookkeeping.el
index 4e2a713399..f2417c89ac 100644
--- a/phps-mode-ast-bookkeeping.el
+++ b/phps-mode-ast-bookkeeping.el
@@ -105,88 +105,107 @@
name))
scope-string))
+;; TODO Should return a list of strings to support the case with a read-only
+;; variable inside a arrow function that should check outer scopes as well
(defun phps-mode-ast-bookkeeping--generate-variable-scope-string
(scope name &optional read-only)
"Generate variable scope string from SCOPE and NAME and optionally
READ-ONLY."
- (let ((scope-string "")
- (namespace))
- (dolist (bubble (reverse scope))
- (let ((scope-type (plist-get bubble 'type))
- (scope-name (plist-get bubble 'name)))
- (cond
-
- ((and
- (equal scope-type 'namespace)
- scope-name)
- (setq
- namespace
- scope-name))
-
- ((and
- (equal scope-type 'class)
- scope-name)
- (if namespace
+ (let ((scope-strings)
+ (bubbles-stack (list (list "" nil (reverse scope))))) ;; scope-string
namespace bubbles
+ (while bubbles-stack
+ (setq
+ bubbles-data
+ (pop bubbles-stack))
+ (let ((scope-string (car bubbles-data))
+ (namespace (car (cdr bubbles-data)))
+ (bubbles (car (cdr (cdr bubbles-data)))))
+ (while bubbles
+ (let* ((bubble (pop bubbles))
+ (scope-type (plist-get bubble 'type))
+ (scope-name (plist-get bubble 'name)))
+ (cond
+
+ ((and
+ (equal scope-type 'namespace)
+ scope-name)
+ (setq
+ namespace
+ scope-name))
+
+ ((and
+ (equal scope-type 'class)
+ scope-name)
+ (if namespace
+ (setq
+ scope-string
+ (format
+ "%s namespace %s class %s"
+ scope-string
+ namespace
+ scope-name))
+ (setq
+ scope-string
+ (format
+ "%s class %s"
+ scope-string
+ scope-name))))
+
+ ((and
+ (equal scope-type 'function)
+ scope-name)
(setq
scope-string
(format
- "%s namespace %s class %s"
+ "%s function %s"
scope-string
- namespace
- scope-name))
- (setq
- scope-string
- (format
- "%s class %s"
- scope-string
- scope-name))))
-
- ((and
- (equal scope-type 'function)
- scope-name)
- (setq
- scope-string
- (format
- "%s function %s"
- scope-string
- scope-name)))
+ scope-name)))
- ((and
- (equal scope-type 'inline-function)
- scope-name)
- (setq
- scope-string
- (format
- "%s anonymous function %s"
- scope-string
- scope-name)))
-
- ((and
- (equal scope-type 'arrow-function)
- scope-name
- (not read-only))
- (setq
- scope-string
- (format
- "%s arrow function %s"
- scope-string
- scope-name)))
-
- ((and
- (equal scope-type 'static)
- (setq
- scope-string
- (format
- "%s static"
- scope-string))))
+ ((and
+ (equal scope-type 'inline-function)
+ scope-name)
+ (setq
+ scope-string
+ (format
+ "%s anonymous function %s"
+ scope-string
+ scope-name)))
+
+ ((and
+ (equal scope-type 'arrow-function)
+ scope-name)
+ (when read-only
+ ;; Branch off here in alternative scope without arrow context
+ ;; but only for read-only contexts
+ (push
+ (list
+ scope-string
+ namespace
+ bubbles)
+ bubbles-stack))
+ (setq
+ scope-string
+ (format
+ "%s arrow function %s"
+ scope-string
+ scope-name)))
- )))
- (setq
- scope-string
- (format
- "%s id %s"
- scope-string
- name))
- scope-string))
+ ((and
+ (equal scope-type 'static)
+ (setq
+ scope-string
+ (format
+ "%s static"
+ scope-string))))
+
+ )))
+ (setq
+ scope-string
+ (format
+ "%s id %s"
+ scope-string
+ name))
+ (push scope-string scope-strings)))
+ scope-strings))
(defun phps-mode-ast-bookkeeping--generate ()
"Generate AST for current buffer."
@@ -214,7 +233,7 @@
(cond
((equal type 'simple-variable)
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
scope
(plist-get item 'name)))
@@ -224,10 +243,11 @@
(plist-get item 'end)))
(defined-p 0))
- (when (gethash id bookkeeping)
- (setq
- defined-p
- 1))
+ (dolist (id ids)
+ (when (gethash id bookkeeping)
+ (setq
+ defined-p
+ 1)))
;; Is a super-global variable?
(when (gethash
@@ -247,7 +267,7 @@
(push `(type function name ,name) sub-scope)
(when-let ((parameter-list (reverse (plist-get item
'parameter-list))))
(dolist (parameter parameter-list)
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
(plist-get parameter 'name)))
@@ -255,10 +275,11 @@
(list
(plist-get parameter 'start)
(plist-get parameter 'end))))
- (puthash
- id
- 1
- bookkeeping)
+ (dolist (id ids)
+ (puthash
+ id
+ 1
+ bookkeeping))
(puthash
object
1
@@ -274,17 +295,18 @@
(push `(type function name ,name) sub-scope)
;; TODO should only do this is method is not static
- (let ((this-id
+ (let ((this-ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
"$this")))
- (puthash
- this-id
- 1
- bookkeeping))
+ (dolist (this-id this-ids)
+ (puthash
+ this-id
+ 1
+ bookkeeping)))
(when-let ((parameter-list (reverse (plist-get item
'parameter-list))))
(dolist (parameter parameter-list)
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
(plist-get parameter 'name)))
@@ -292,10 +314,11 @@
(list
(plist-get parameter 'start)
(plist-get parameter 'end))))
- (puthash
- id
- 1
- bookkeeping)
+ (dolist (id ids)
+ (puthash
+ id
+ 1
+ bookkeeping))
(puthash
object
1
@@ -398,7 +421,7 @@
(push `(,scope ,child) bookkeeping-stack))))
((equal type 'assign-property-variable)
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
scope
(plist-get item 'key)))
@@ -407,14 +430,16 @@
(plist-get item 'start)
(plist-get item 'end)))
(defined 1))
- (when-let ((predefined (gethash id bookkeeping)))
- (setq
+ (dolist (id ids)
+ (when-let ((predefined (gethash id bookkeeping)))
+ (setq
+ defined
+ (1+ predefined))))
+ (dolist (id ids)
+ (puthash
+ id
defined
- (1+ predefined)))
- (puthash
- id
- defined
- bookkeeping)
+ bookkeeping))
(puthash
object
defined
@@ -457,7 +482,7 @@
((and
(equal type 'assign-variable)
(plist-get (plist-get item 'key) 'name))
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
scope
(plist-get (plist-get item 'key) 'name)))
@@ -466,14 +491,16 @@
(plist-get (plist-get item 'key) 'start)
(plist-get (plist-get item 'key) 'end)))
(defined 1))
- (when-let ((predefined (gethash id bookkeeping)))
- (setq
+ (dolist (id ids)
+ (when-let ((predefined (gethash id bookkeeping)))
+ (setq
+ defined
+ (1+ predefined))))
+ (dolist (id ids)
+ (puthash
+ id
defined
- (1+ predefined)))
- (puthash
- id
- defined
- bookkeeping)
+ bookkeeping))
(puthash
object
defined
@@ -494,11 +521,11 @@
static-p
t))))
(if (stringp subject)
- (let ((id))
+ (let ((ids))
(when static-p
(push '(type static) sub-scope))
(setq
- id
+ ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
subject))
@@ -508,14 +535,16 @@
(plist-get item 'end)))
(defined 1))
;; (message "id: %S from %S" id item)
- (when-let ((predefined (gethash id bookkeeping)))
- (setq
+ (dolist (id ids)
+ (when-let ((predefined (gethash id bookkeeping)))
+ (setq
+ defined
+ (1+ predefined))))
+ (dolist (id ids)
+ (puthash
+ id
defined
- (1+ predefined)))
- (puthash
- id
- defined
- bookkeeping)
+ bookkeeping))
(puthash
object
defined
@@ -544,14 +573,15 @@
((equal type 'catch)
(when-let ((optional-variable
(plist-get item 'optional-variable)))
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
scope
optional-variable)))
- (puthash
- id
- 1
- bookkeeping)
+ (dolist (id ids)
+ (puthash
+ id
+ 1
+ bookkeeping))
(puthash
(list
(plist-get item 'optional-variable-start)
@@ -579,7 +609,8 @@
bookkeeping)
(let* ((sub-scope (cdr scope))
- (variable-id
+ (predefined)
+ (variable-ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
(concat "$" property-name)))
@@ -592,9 +623,16 @@
(plist-get item 'property-start)
(plist-get item 'property-end))))
;; (message "dereferenced: %S %S" variable-id symbol-id)
- (if (or
- (gethash variable-id bookkeeping)
- (gethash symbol-id bookkeeping))
+ (when (gethash symbol-id bookkeeping)
+ (setq
+ predefined
+ t))
+ (dolist (variable-id variable-ids)
+ (when (gethash variable-id bookkeeping)
+ (setq
+ predefined
+ t)))
+ (if predefined
(puthash
bookkeeping-object
1
@@ -622,7 +660,8 @@
((equal member-type 'simple-variable)
(let ((sub-scope (cdr scope)))
(push '(type static) sub-scope)
- (let ((variable-id
+ (let ((predefined)
+ (variable-ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
(plist-get member 'name)))
@@ -630,7 +669,12 @@
(list
(plist-get member 'start)
(plist-get member 'end))))
- (if (gethash variable-id bookkeeping)
+ (dolist (variable-id variable-ids)
+ (when (gethash variable-id bookkeeping)
+ (setq
+ predefined
+ t)))
+ (if predefined
(puthash
bookkeeping-object
1
@@ -656,7 +700,7 @@
(push `(,sub-scope ,inner-statement) bookkeeping-stack)))
(when-let ((parameter-list (plist-get item 'parameter-list)))
(dolist (parameter parameter-list)
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
(plist-get parameter 'name)))
@@ -664,10 +708,11 @@
(list
(plist-get parameter 'start)
(plist-get parameter 'end))))
- (puthash
- id
- 1
- bookkeeping)
+ (dolist (id ids)
+ (puthash
+ id
+ 1
+ bookkeeping))
(puthash
object
1
@@ -682,7 +727,7 @@
(push `(,sub-scope ,inner-statement) bookkeeping-stack)))
(when-let ((parameter-list (plist-get item 'parameter-list)))
(dolist (parameter parameter-list)
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
(plist-get parameter 'name)))
@@ -690,17 +735,18 @@
(list
(plist-get parameter 'start)
(plist-get parameter 'end))))
- (puthash
- id
- 1
- bookkeeping)
+ (dolist (id ids)
+ (puthash
+ id
+ 1
+ bookkeeping))
(puthash
object
1
bookkeeping))))
(when-let ((lexical-vars (plist-get item 'lexical-vars)))
(dolist (lexical-var lexical-vars)
- (let ((id
+ (let ((ids
(phps-mode-ast-bookkeeping--generate-variable-scope-string
sub-scope
(plist-get lexical-var 'name)))
@@ -708,10 +754,11 @@
(list
(plist-get lexical-var 'start)
(plist-get lexical-var 'end))))
- (puthash
- id
- 1
- bookkeeping)
+ (dolist (id ids)
+ (puthash
+ id
+ 1
+ bookkeeping))
(puthash
object
1
diff --git a/test/phps-mode-test-ast.el b/test/phps-mode-test-ast.el
index 77332896c2..a3447172ac 100644
--- a/test/phps-mode-test-ast.el
+++ b/test/phps-mode-test-ast.el
@@ -869,6 +869,9 @@
(phps-mode-test--hash-to-list phps-mode-ast-bookkeeping--index t)
'((" id $a" 1) ((8 10) 1) (" id $b" 1) ((13 15) 1) (" id $c" 1) ((18
20) 1) ((31 33) 1) ((51 53) 1) ((99 101) 1) ((119 121) 1) ((167 169) 1) ((187
189) 1))))))
+ ;; TODO Test this and make sure all variables hit
+ ;; class myClass { private $test = 'abc'; public function test($d) { return
fn($d) => $this->test . $d; }} $a = new myClass(); echo $a->test('def')('ghi');
+
(message "\n-- Ran tests for bookkeeping generation. --"))
(defun phps-mode-test-ast ()
- [elpa] externals/phps-mode ff62626f48 110/212: Passing indent test for square bracket array, (continued)
- [elpa] externals/phps-mode ff62626f48 110/212: Passing indent test for square bracket array, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 50d006852c 129/212: Fixed issue with indent being dependent on position of point on current line, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 3de8eff9f1 114/212: More wrestling with indentation, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode d3b5a85b70 001/212: Using property list as basis for AST, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode a80c12ef14 005/212: Passing imenu-generation for file with multiple namespaces, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 0071b40aae 007/212: Syntax coloring is now in separate file, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 1cc4aed616 016/212: Passing first bookkeeping test via generation via parser SDT, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode b3b27951bd 015/212: More work on bookkeeping, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 395a4ed045 036/212: Started on bookkeeping via parser SDT for properties, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 69341c9709 039/212: More working on bookkeeping of class properties via parser SDT, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode cefab36fa8 054/212: More work on bookkeeping arrow function variables,
Christian Johansson <=
- [elpa] externals/phps-mode d9cbe630b3 068/212: Passing bookkeeping of variables inside echo statements, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 0919855a31 073/212: Added SDT for concat operation, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 5df554fc47 070/212: Bookkeeping via AST properly handles $this reference in only non-static methods, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 2432928a86 096/212: Indentation passing more tests for alternative control structures, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode cef879268b 103/212: Added two new tests for failing indentation for multi-line concatenated strings, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 40ce934c11 116/212: Indent support for string-doc in assignment, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode fbe600500f 135/212: More work on array and argument element indentation, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 6cb18913e4 152/212: Improved indentation for lines ending with closing bracket and comma, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 26d7b64086 144/212: Added TODO item for indentation, Christian Johansson, 2022/01/26
- [elpa] externals/phps-mode 0e7eb04800 149/212: Passing indentation test for multiple function arguments with default values, Christian Johansson, 2022/01/26