On Fri Nov 29 12:17:54 2024 +0000, Fabian Maurer wrote:
Looks correct to me, but IMHO a test would be useful. We also still seem to have an off-my-one error in that function, I noticed while testing your changes. If you want, I'd offer to create a bunch of tests, in hopes of saving you the work?
I've pushed new version with additional fixes and tests. Thank you.