Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misc minor issues (courtesy of Stefan Monnier) #193

Merged
merged 2 commits into from
Jan 18, 2025

Conversation

josteink
Copy link
Member

  • typescript-mode.el: Activate lexical-binding.
  • (typescript--font-lock-keywords-2): Use backquote. Quote face names so as not to rely on obsolete font-lock-*-face variables.
  • (typescript--re-search-forward, typescript--re-search-backward): Use funcall instead of eval to save kittens and be compatible with lexical-binding.
  • (typescript--font-lock-keywords-3): Quote face names as above.
  • (typescript--font-lock-keywords-4): Fix "[]" to "\[\]".
  • Use font-lock-function-call-face for function calls.
  • Quote face names as above.

* typescript-mode.el (typescript--function-face): New function.
(typescript--font-lock-keywords-4): Use `font-lock-function-name-face`
only for definitions, not calls and not decorators.

* typescript-mode-general-tests.el (font-lock/level-four)
(font-lock/method-call-with-keyword-name): Adjust to new faces.
(font-lock/funargs--method--single-line--with-type): Also test
the face of the method definition.
* typescript-mode.el: Activate `lexical-binding`.
(typescript--font-lock-keywords-2): Use backquote.  Quote face names
so as not to rely on obsolete `font-lock-*-face` variables.
(typescript--re-search-forward, typescript--re-search-backward):
Use `funcall` instead of `eval` to save kittens and be compatible with
`lexical-binding`.
(typescript--font-lock-keywords-3): Quote face names as above.
(typescript--font-lock-keywords-4): Fix "\[\]" to "\\[\\]".
Quote face names as above.

* .gitignore: Add ELPA-generated files.
@josteink josteink force-pushed the feature/various-fixes branch from 3e7cc81 to 6f3b8b4 Compare January 18, 2025 20:15
@josteink josteink merged commit 481df3a into master Jan 18, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants