Re: [frogs] PATCH for getting output-suffix to work (Trackers 714 and 404)

[ Thread Index | Date Index | More lilynet.net/frogs Archives ]


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am Donnerstag, 6. August 2009 18:20:26 schrieb Ian Hulin:
>  Here's the first stage of work on these trackers,  I've taken the work
> Marek Klein did and polished it a little bit. 

Great to see that this is not forgotten (like so many other not yet finished 
patches)! Some comments below...

> +    (if (string? output-suffix)
> +       (set! result (format "~a-~a" base (string-regexp-substitute
> +                                        "[^a-zA-Z0-9-]" "_" output-suffix))))

This rules out any accented or other foreigh letters! E.g. all chinese suffixes 
will simply turn into ______. Also, German instrument names like "Flöte" or 
"Hörner" would become "Fl_te" and "H_rner" in the suffix.

I supose it would be better to use 
  "[^[:alnum:]-+]"
as the regexp to allow all alpha-numeric letters.

>  (define-public (version-not-seen-message input-file-name)
>    (ly:message
> -   "~a:0: ~a: ~a"
> +   "~a:0: ~a ~a"
>      input-file-name
> -    (_ "warning: ")
> +    (_ "warning:")

Nice catch, but has nothing to do with output-suffix, so it should go into its 
own patch.

Cheers,
Reinhold
- -- 
- ------------------------------------------------------------------
Reinhold Kainhofer, reinhold@xxxxxxxxxxxxx, http://reinhold.kainhofer.com/
 * Financial & Actuarial Math., Vienna Univ. of Technology, Austria
 * http://www.fam.tuwien.ac.at/, DVR: 0005886
 * LilyPond, Music typesetting, http://www.lilypond.org
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iD8DBQFKewkITqjEwhXvPN0RAjYEAJ0TaV392KlkU//Xo+iQ45NDN4cIoACfeugc
7Xe8HmcDYvOtpTpWso0nDug=
=Fa8L
-----END PGP SIGNATURE-----

---

----
Join the Frogs!


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/