Skip to content

Conversation

@axvr
Copy link
Member

@axvr axvr commented Jun 15, 2021

Syntax highlight missing symbols and improve consistency of syntax groups.

To do

  • Fix mis-grouping of print-method and print-dup.
  • Add case* to clojureCond.
  • Fix linter errors/warnings.
  • Refactor code.
  • Complete TODOs?

Current changes

(Note: items suffixed with (new) were never syntax highlighted by Clojure.vim or Vim-clojure-static.)

clojureCond

+ if
+ if*    (new)
+ case*  (new)

clojureSpecial

+ &
+ reify*  (new)
- if
+ letfn*  (new)
+ clojure.core/import*  (new)
- recur
- loop
+ let*    (new)
+ fn*     (new)
+ letfn
- def

clojureRepeat

+ loop*   (new)
+ loop
+ recur

clojureMacro

- letfn

clojureDefine

+ deftype*  (new)
+ def

@SevereOverfl0w
Copy link
Member

https://github.com/clojure-emacs/orchard/blob/master/src/orchard/meta.clj some code here might be useful for what you're doing (based on a quick peek at the PR). Maybe you've covered all the relevant cases already though!

@axvr axvr force-pushed the missing-syntax branch 3 times, most recently from 42e64a8 to 24597e6 Compare January 15, 2022 19:39
@axvr axvr changed the title Experimental changes to add missing syntax and correct highlight groups Syntax highlight missing symbols and improve consistency of syntax groups Feb 4, 2022
@axvr axvr marked this pull request as ready for review February 4, 2022 23:16
@axvr axvr changed the title Syntax highlight missing symbols and improve consistency of syntax groups Syntax highlight more symbols and make grouping more consistent Feb 4, 2022
@axvr axvr force-pushed the missing-syntax branch 2 times, most recently from 2f1f4e9 to 4a8a3fe Compare February 4, 2022 23:34
@axvr axvr merged commit 0057222 into master Feb 6, 2022
@axvr axvr deleted the missing-syntax branch February 6, 2022 22:17
@axvr axvr added the highlighting Affects syntax highlighting label Jun 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

highlighting Affects syntax highlighting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants