On 2018/01/01 17:36:41, lemzwerg wrote:

https://codereview.appspot.com/339090043/diff/80001/ps/encodingdefs.ps#newcode721
ps/encodingdefs.ps:721: /.notdef /.notdef /.notdef /.notdef /.notdef
Yes, the encoding vector should contain exactly 256 elements.

This isn’t the case in other vectors too, I’ll fix that with the next
patch set.


https://codereview.appspot.com/339090043/diff/80001/ps/encodingdefs.ps#newcode860
ps/encodingdefs.ps:860: /nine {<7c> show} def
The order in /LilyOtherEncoding doesn't matter.  It's this very
encoding vector
that maps glyph names to glyph indices.

Ok. Looking at the resulting otf, type1, and svg fonts it doesn’t change
the encoding because “n” sits at its ASCII code, so I believe it’s ok to
shift the indices.

https://codereview.appspot.com/339090043/
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to