Re: [PATCH] Fix crash when output-preview-framework is missing

[Reinhold Kainhofer]

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

Am Donnerstag, 28. Mai 2009 22:18:23 schrieb Neil Puttock:
> +     warning (_f ("the `%s' backend does not support -dprint-pages",
> +                  get_output_backend_name ()));
>
> +     warning (_f ("the `%s' backend does not support -dpreview",
> +                  get_output_backend_name ()));
>
> I'm not sure about these; personally I'd prefer having the important
> information (the unsupported backend) at the end of the message:
>
> "unsupported backend for program option -dpreview: `%s'"

This way, the explanation "unsupported backend" and the type of backend are 
very far apart and as such this message is much harder to understand. If you 
prefer the backend at the end, I would formulate it as:

Program option -dpreview not supported by backend `%s'.

Cheers,
Reinhold
- -- 
- ------------------------------------------------------------------
Reinhold Kainhofer, address@hidden, 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)

iD8DBQFKHvgTTqjEwhXvPN0RAuT3AJ92vRnthU1m5vIwJ2aDykie6CW5twCgrbxI
+EUFD51kZliKvnrI/OscsZo=
=Rbmb
-----END PGP SIGNATURE-----