(In reply to comment #13)
> - tv.tv_sec = 0;
> + tv.tv_sec = 5;
> tv.tv_usec = kClipboardTimeout;
>
> You're adding 5 seconds here? wouldn't it make more sense to change
> kClipboardTimeout?
Well, because increasing the microseconds is not enough, you need a timeout
of several seconds for large images.
> + if (gdk_pixbuf_get_has_alpha(new_pixbuf) == TRUE)
> + format = imgIEncoder::INPUT_FORMAT_RGBA;
>
> Why bother since we can't create a transparent image anyway?
We can't? So why does ImageEncoder provide that format flag?
(In reply to comment #13)
> - tv.tv_sec = 0;
> + tv.tv_sec = 5;
> tv.tv_usec = kClipboardTimeout;
>
> You're adding 5 seconds here? wouldn't it make more sense to change
> kClipboardTimeout?
Well, because increasing the microseconds is not enough, you need a timeout
of several seconds for large images.
> + if (gdk_pixbuf_ get_has_ alpha(new_ pixbuf) == TRUE) :INPUT_ FORMAT_ RGBA;
> + format = imgIEncoder:
>
> Why bother since we can't create a transparent image anyway?
We can't? So why does ImageEncoder provide that format flag?
Ciao,
Tobias