[slime-devel] Improved slime-global-variable-name-p

[Christian Lynbech]

>>>>> "Lynn" == Lynn Quam <quam at ai.sri.com> writes:

Lynn> I recommend the following change to SLIME-GLOBAL-VARIABLE-NAME-P
Lynn> so that package-qualified global variables are recognized.

Lynn> (defun slime-global-variable-name-p (name)
Lynn>   "Is NAME a global variable?
Lynn> Globals are recognised purely by *this-naming-convention*."
Lynn>   ;(string-match "^\\*.*\\*$" name)
Lynn>   ;; LHQ Sat Nov 27 2004 -- allow optional <package-name>: or <package-name>::
Lynn>   (string-match "^\\(.*::?\\)?\\*.*\\*$" name)
Lynn>   )

I am a bit unsure where `slime-global-variable-name-p' is actually
used but shouldn't we include the global constant naming convention
such that we have something like this:

        (string-match "^\\(.*::?\\)?[*+].*[*+]$" name)

instead? (This will of course also match "+name*" and vice versa but I
see no harm in that).


------------------------+-----------------------------------------------------
Christian Lynbech       | christian #\@ defun #\. dk
------------------------+-----------------------------------------------------
Hit the philistines three times over the head with the Elisp reference manual.
                                        - petonic at hal.com (Michael A. Petonic)