Mirror of forum.nim-lang.org

8640 :: Pygments Nim lexer update?

• matkuki (orginal) [2021-11-18T22:20:08+01:00] view original

  Hi guys,

  Does anyone know enough about Pygments (https://github.com/pygments/pygments) to update the Nim/Nimrod lexer for it?

  Thanks, Matic

---

• shirleyquirk (orginal) [2021-11-19T01:11:08+01:00] view original

  What's wrong with it? Is there something in particular that's broken?

---

• Vindaar (orginal) [2021-11-19T09:13:15+01:00] view original

  Add to this unicode symbols are broken. Anything after in the line after a unicode symbol has a wrong color.

  So yeah, I'd say there's some stuff that could be fixed.

  As far as I can tell the code is pretty simple though. The Nim related logic is here:

  https://github.com/pygments/pygments/blob/master/pygments/lexers/nimrod.py

---

• matkuki (orginal) [2021-11-19T10:34:45+01:00] view original

  Right, some stuff doesn't work or is missing:

  • multiline comments (normal/doc)
  • triple quoted string
  • proc/func names are not colored
  • case statement branches could also be colored

  I'll see if I can upgrade it. Thanks for the information.

---

• matkuki (orginal) [2022-09-24T21:58:42+02:00] view original

  Nim lexer for Pygments updated: https://github.com/pygments/pygments/pull/1970

---

• Yardanico (orginal) [2022-09-24T23:27:05+02:00] view original

  Was there a reason as to why you changed some of the echo to be Echo? Maybe you got confused by the Nim syntax a bit, but in Nim echo "hello" and echo("hello") mean the exact same thing and call the exact same built-in proc. Echo("something") wouldn't work because Nim is case-sensitive to the first character, so echo != Echo, and generally people don't name their procedures in PascalCase.

---

• matkuki (orginal) [2022-09-24T23:42:33+02:00] view original

  If you check the git log, there was nothing changed in the example Nim code files example.nim and test.nim. This is a left-over from the Nimrod days, when there was full case insensitivity. They even still call it NimrodLexer for compatibility.

---