[prev in list] [next in list] [prev in thread] [next in thread] 

List:       lilypond-bug
Subject:    Re: glyph-path-regexp in SVG output is overly strict
From:       Jonas Hahnfeld via bug-lilypond <bug-lilypond () gnu ! org>
Date:       2020-02-29 10:08:24
Message-ID: 6c7be9923409532400cc5f830b8e8638de2b542d.camel () hahnjo ! de
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Am Freitag, den 21.02.2020, 12:58 +0100 schrieb Jonas Hahnfeld via bug-
lilypond:
> Am Freitag, den 21.02.2020, 11:33 +0000 schrieb Simon Tatham:
> > Jonas Hahnfeld <
> > hahnjo@hahnjo.de
> > 
> > > wrote:
> > > looking at 
> > > https://sourceforge.net/p/testlilyissues/issues/5779/
> > > 
> > >  and
> > > the grammar you linked, I think the regular expression is also missing
> > > '+Aa,'. Does it make sense to fuse attached patch?
> > 
> > The idea looks sensible to me, but I think you've made a mistake in the
> > regular expression syntax. The character '-' has to appear immediately
> > after the open bracket, otherwise it's taken to be the punctuation in a
> > range such as "A-Z". So the character class
> > [+-MmZzLlHhVvCcSsQqTtAa0-9,.Ee\n ] will match (at least) any character
> > between '+' and 'M'.
> > 
> > If you start the brackets with [-+Mm...] instead of [+-Mm...] then I
> > think it does what you intended.
> 
> Absolutely correct, good spot. Luckily (for my testing) '-' is between
> '+' and 'M', but that's actually too broad. Will fix the patch.

Hi Simon,

the patches are now in master (actually since Tuesday). Additionally
David has applied them to stable/2.20 and they will be part of the
imminent stable release 2.20!
Thanks again for your contribution and the concise descriptions.

Regards,
Jonas

["signature.asc" (application/pgp-signature)]
[Attachment #6 (text/plain)]

_______________________________________________
bug-lilypond mailing list
bug-lilypond@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-lilypond


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic