mirror of
https://github.com/golang/go
synced 2024-11-11 23:20:24 -07:00
misc/emacs: do not highlight built-in function if not followed by '('
Name of built-in function is not reserved word in Go, and you can use it as variable name. "new" is often used as local variable, for instance. This patch is to apply font-lock-builtin-face only when built-in function name is followed by '(', so that it doesn't highlight non-function variable that happen to have the same name as built-in function. LGTM=dominik.honnef R=golang-codereviews, dominik.honnef, adonovan CC=golang-codereviews https://golang.org/cl/79260043
This commit is contained in:
parent
0f82cfd3f0
commit
5f5e280e14
@ -262,7 +262,7 @@ For mode=set, all covered lines will have this weight."
|
||||
;; doesn't understand that
|
||||
(append
|
||||
`((,(go--regexp-enclose-in-symbol (regexp-opt go-mode-keywords t)) . font-lock-keyword-face)
|
||||
(,(go--regexp-enclose-in-symbol (regexp-opt go-builtins t)) . font-lock-builtin-face)
|
||||
(,(concat "\\(" (go--regexp-enclose-in-symbol (regexp-opt go-builtins t)) "\\)[[:space:]]*(") 1 font-lock-builtin-face)
|
||||
(,(go--regexp-enclose-in-symbol (regexp-opt go-constants t)) . font-lock-constant-face)
|
||||
(,go-func-regexp 1 font-lock-function-name-face)) ;; function (not method) name
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user