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

[Neil Puttock]

2009/5/28 Patrick McCarty <address@hidden>:

> Attached is a revised patch.

That was quick. :)

LGTM.

+       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'"

Regards,
Neil