On 13.11.2012 13:06, Christophe Fergeau wrote:
> When falling back to saving to .png, the filename might not
> end with .png. This commit appends the .png extension to the
> screenshot filename if it's missing.
> ---
> src/virt-viewer-window.c | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/src/virt-viewer-window.c b/src/virt-viewer-window.c
> index aec51ae..1c2a63b 100644
> --- a/src/virt-viewer-window.c
> +++ b/src/virt-viewer-window.c
> @@ -801,8 +801,16 @@ virt_viewer_window_save_screenshot(VirtViewerWindow *self,
>
> if (format == NULL) {
> g_debug("unknown file extension, falling back to png");
> - gdk_pixbuf_save(pix, file, "png", NULL,
> - "tEXt::Generator App", PACKAGE, NULL);
> + if (!g_str_has_suffix(file, ".png")) {
> + char *png_filename;
> + png_filename = g_strconcat(file, ".png", NULL);
> + gdk_pixbuf_save(pix, png_filename, "png", NULL,
> + "tEXt::Generator App", PACKAGE, NULL);
> + g_free(png_filename);
> + } else {
> + gdk_pixbuf_save(pix, file, "png", NULL,
> + "tEXt::Generator App", PACKAGE, NULL);
> + }
> } else {
> char *type = gdk_pixbuf_format_get_name(format);
> g_debug("saving to %s", type);
>
ACK
Michal
