Comment 14 for bug 241276

Revision history for this message
In , Tokoe (tokoe) wrote :

(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?

Ciao,
Tobias