Sorry for the false alarm. Reverting this patch does not appear to make a
discernible difference to the time of running make doc, so this apparent recent
increase in compilation time does not seem related to this patch. I'll have to
look at possible other reasons for what appears to be a loss of performance.
For the record, here with patch operative:
real 42m2.760s
user 121m45.305s
sys 18m53.051s
and here with patch reverted:
real 42m15.157s
user 122m7.991s
sys 18m53.564s
There was light usage in either case so the slight difference here does not say
anything worthwhile.
---
** [issues:#5741] Parse inline scheme using per-expression port**
**Status:** Fixed
**Labels:** Fixed_2_21_0
**Created:** Fri Feb 07, 2020 02:52 PM UTC by Han-Wen Nienhuys
**Last Updated:** Sat Feb 22, 2020 02:28 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 [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto