[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #5741 Parse inline scheme using per-expression port

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

That would be indicative of the closure mechanism not working. The way the closure mechanism is implemented means that if input locations of # inside of #{ ... #} are not exactly matched to the recorded locations, the interpretation reverts to evaluation outside of a captured environment, and "mag" is not defined there. So it could again be related to the "off-by-1" error Han-Wen stipulated as still afflicting the patch.

---

[issues:#5741] Parse inline scheme using per-_expression_ port

Status: Started
Created: Fri Feb 07, 2020 02:52 PM UTC by Han-Wen Nienhuys
Last Updated: Mon Feb 10, 2020 03:11 PM UTC
Owner: Han-Wen Nienhuys

Introduces a throw-away string port, Overlay_string_port, which makes
a port out of a random section of a C string. It does not own the
string, so there is no overhead in creating and discarding the ports
during parse time.

For GUILE v2, Overlay_string_port uses UTF-8 encoding, so UTF-8
encoded string constants within the Scheme constants are interpreted
as Unicode correctly.

This obviates the string port that Source_file carries along.

This commit fixes a problem with GUILE 2.2.6, where LilyPond
calculates offsets in the source file as bytes, while GUILE interprets
the source file as UTF-8 encoded Unicode. As a result, files with
Unicode before embedded scheme break completely.

https://codereview.appspot.com/557330043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto