Re: [PATCH] add SRFI-119 / language/wisp to Guile? (new patch with more tests, squashed)

[Ludovic Courtès]

Hi Arne,

"Dr. Arne Babenhauserheide" <arne_bab@web.de> skribis:

> From 81e7cbbade4fd01e092a2c39a3af90e4abf7f684 Mon Sep 17 00:00:00 2001
> From: Arne Babenhauserheide <arne_bab@web.de>
> Date: Mon, 11 Mar 2024 06:34:52 +0100
> Subject: [PATCH] Add language/wisp, Wisp tests, and SRFI-119 documentation
>
> * doc/ref/srfi-modules.texi (srfi-119): add node
> * module/language/wisp.scm: New file.
> * module/language/wisp/spec.scm: New file.
> * test-suite/tests/srfi-119.test: New file.

I have the pleasure to inform you that I have finally pushed this!  :-)

Apologies for taking so long, and thank you for being patient.

Some of the suggestions I made earlier¹ were still not implemented
though:

  1. Using uninterned symbols rather than UUIDs.

  2. Using a record type for lines instead of tuples.

  3. Avoiding source properties.

I took the liberty to implement #1 in commit
27feb2bfd38087cf03989673da0fc74ed795307d.  Tests pass but please let me
know if you notice something wrong!

It’d be great if you could look at #2 and #3 along the lines of what I
suggested earlier.  This time we should be able to move forward more
quickly.  :-)

Thanks!

Ludo’.

¹ https://lists.gnu.org/archive/html/guile-devel/2023-08/msg00009.html