Skip to content

Commit

Permalink
tagging: improve comment
Browse files Browse the repository at this point in the history
  • Loading branch information
avodonosov authored Jun 10, 2023
1 parent a97c8b5 commit 7a61fd7
Showing 1 changed file with 17 additions and 15 deletions.
32 changes: 17 additions & 15 deletions queries/tags.scm
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,8 @@
"(?i)^(cl:)?(let|let\\*|flet|labels|macrolet|symbol-macrolet)$")
)

;; TODO: exclude also:
;; TODO:
;; - exclude also:
;; - (defclass name (parent parent2)
;; ((slot1 ...)
;; (slot2 ...))
Expand All @@ -49,24 +50,25 @@
;; as a function call - exclude it too
;; - the same for labels
;; - the same macrolet
;; - (flet ((func-1 (param1 param2))) ...)
;; - instead of simply excluding it, as we do today,
;; tag func-1 as @local.definition.function (I suppose)
;; - the same for labels, macrolet
;; - @local.scope for let, let*, flet, labels, macrolet
;; - I guess the whole span of the scope text,
;; till the closing paren, should be tagged as @local.scope;
;; Hopefully, combined with @local.definition.function
;; withing the scope, the usual @reference.call within
;; that scope will refer to the local definition,
;; and there will be no need to use @local.reference.call
;; (which is more difficult to implement).
;; - When implementing, remeber the scope rules differences
;; of let vs let*, flet vs labels.
;; - what else?
;; (that's a non-goal to completely support all macros
;; and special operators, but every one we support
;; makes the solution a little bit better)
;; - (flet ((func-1 (param1 param2))) ...)
;; - instead of simply excluding it, as we do today,
;; tag func-1 as @local.definition.function (I suppose)
;; - the same for labels, macrolet
;; - @local.scope for let, let*, flet, labels, macrolet
;; - I guess the whole span of the scope text,
;; till the closing paren, should be tagged as @local.scope;
;; Hopefully, combined with @local.definition.function
;; withing the scope, the usual @reference.call within
;; that scope will refer to the local definition,
;; and there will be no need to use @local.reference.call
;; (which is more difficult to implement).
;; - When implementing, remeber the scope rules differences
;; of let vs let*, flet vs labels.


;; Inlclude all other cases - list literal with symbol as the
;; first element
Expand Down

0 comments on commit 7a61fd7

Please sign in to comment.