On Mon, Mar 12, 2012 at 02:11:33PM +0100, Marc-André Lureau wrote: > Dump an image in PNG format if the URI ends with ".png" and PNG > support is enabled. > --- > hw/vga.c | 75 > +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 74 insertions(+), 1 deletions(-) > > diff --git a/hw/vga.c b/hw/vga.c > index 9f7ca89..0210cde 100644 > --- a/hw/vga.c > +++ b/hw/vga.c > @@ -32,6 +32,13 @@ > #include "xen.h" > #include "qemu_socket.h" > > +#ifdef CONFIG_PNG > +/* The following define is needed by pngconf.h. Otherwise it won't compile, > + because setjmp.h was already included by qemu-common.h. */ > +#define PNG_SKIP_SETJMP_CHECK > +#include <png.h> > +#endif
[snip] I'd be inclined to just switch over to use gdk-pixbuf for saving images. It copes with a wide array of formats, and is a natrual fit now that we're using GLib. NB, gdk-pixbuf is distributed and built separate from GDK/GTK, so this wouldn't create a hard dep on GTK - you could still build with just the gdk-pixbuf support. While allowing gdk-pixbuf to guess format based off the file name is fine, I'd suggest we should add an explicit format argument to the 'screendump' command. If omitted, the format would be guessed from filename if possible. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
