git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] diff: "lisp" userdiff_driver

From
Scott L. Burson <scott@sympoiesis.com>
Date
Nov 27, 2025, 02:10 UTC
Message-ID
<CAF5LJ4AgJvMHej1uRm5Q0v_BTavmm+aXPBC-nGmBp9URX16Gkw@mail.gmail.com>
In-Reply-To
<CALnO6CBFKjewrkPeEUh7Q-A2dZ7Fknjy4DszG8xCKu-NvGETfQ@mail.gmail.com>
On Thu, Nov 20, 2025 at 8:47 AM D. Ben Knoble <ben.knoble@gmail.com> wrote:
Show 7 quoted lines
> Without "going there," I think there are enough differences to warrant
> a different driver. (OTOH, I have sometimes wanted to teach the Scheme
> driver that most "def" things are probably definitions.) Our
> indentation is less rigid in that indented forms may be more deeply
> nested than only one or two spaces (and we of course have more
> definition forms than "only things starting with def"), and I don't
> understand the downthread desire to not permit tabs.

The proposal on the table is not to replace the current Scheme regexp with the Lisp one, but rather to disjoin them, so as to match any line that either one would match. So you don't need to worry about false negatives.

The thing about tabs was that, the way I had initially written the regexp, it would have matched a line starting with one or two tabs, or a tab and a space -- but no other whitespace -- followed by "(def". This was not my intention. I was just trying to match lines starting with a space or two and "(def". Again, this is in addition to the lines currently being matched by the Scheme regexp.

> As for Scheme community, I'll suggest asking on the Racket channels
> (Discourse is probably best if you want a mailing-list-like
> discussion?)

I have asked on Reddit under /r/scheme, and got only the same objection that you offered ("define" etc. forms more deeply nested), with the same resolution (disjoining the new pattern to the existing one).

Reddit says my post got some 4300 views. That seems like an adequate sample to me, but I can try elsewhere if you think it's important.

I have an updated version of the patch ready; I will submit it shortly.

Previous: D. Ben KnobleNext: Junio C Hamano
Message 5 of 27 in “diff: "lisp" userdiff_driver”
  1. diff: "lisp" userdiff_driverScott L. Burson via GitGitGadget, Nov 15, 2025
  2. Johannes SixtNov 15, 2025
  3. Scott L. BursonNov 15, 2025
  4. D. Ben KnobleNov 20, 2025
  5. Scott L. BursonNov 27, 2025
  6. Junio C HamanoNov 16, 2025
  7. Scott L. BursonNov 17, 2025
  8. Junio C HamanoNov 18, 2025
  9. 0/2 userdiff: extend Scheme support to cover other Lisp dialectsScott L. Burson via GitGitGadget, Nov 27, 2025
  10. 1/2 diff: "lisp" userdiff_driverScott L. Burson via GitGitGadget, Nov 27, 2025
  11. Scott L. BursonNov 27, 2025
  12. Johannes SixtNov 27, 2025
  13. 2/2 merge with Scheme regexp; fix bugsScott L. Burson via GitGitGadget, Nov 27, 2025
  14. Johannes SixtNov 27, 2025
  15. Johannes SixtDec 2, 2025
  16. Scott L. BursonJan 14, 2026
  17. Johannes SixtJan 14, 2026
  18. 0/2 userdiff: extend Scheme support to cover other Lisp dialectsScott L. Burson via GitGitGadget, Jan 15, 2026
  19. 1/2 userdiff: tighten word-diff test case of the scheme driverJohannes Sixt via GitGitGadget, Jan 15, 2026
  20. 2/2 userdiff: extend Scheme support to cover other Lisp dialectsScott L. Burson via GitGitGadget, Jan 15, 2026
  21. Johannes SixtJan 16, 2026
  22. Scott L. BursonJan 17, 2026
  23. Johannes SixtJan 17, 2026
  24. 0/2 userdiff: extend Scheme support to cover other Lisp dialectsScott L. Burson via GitGitGadget, Apr 15, 2026
  25. 1/2 userdiff: tighten word-diff test case of the scheme driverJohannes Sixt via GitGitGadget, Apr 15, 2026
  26. 2/2 userdiff: extend Scheme support to cover other Lisp dialectsScott L. Burson via GitGitGadget, Apr 15, 2026
  27. Johannes SixtApr 15, 2026

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.