Looks fine to me.

Thanks,
Alexander.

On 31/05/16 17:29, Sergey Bylokhov wrote:
Hi, Mario.
Thanks for your contribution! there is tiny typo in the native: isGtkSupported should be useGtk. It will be good if someone run jprt job to confirm that the build is ok, since make file was changed.

On 31.05.16 13:45, Semyon Sadetsky wrote:


On 5/31/2016 1:36 PM, Mario Torre wrote:
2016-05-30 17:53 GMT+02:00 Semyon Sadetsky <semyon.sadet...@oracle.com>:

The rest is OK to me.
Great, thanks!

This is the final webrev:

http://cr.openjdk.java.net/~neugens/8150954/webrev.04/
Thanks a lot!

Assuming it's still OK for you, I believe I need another reviewer's OK
to push?
Yes, reviewing is not that fast currently. Probably azvegint could help,
he is an author of the GTK screenshoting code. I'll ping him.

--Semyon

Cheers,
Mario




Reply via email to