Re: [8u/9] Review request 8156116 : [macosx] two JNI locals to delete in AWTWindow.m, CGraphicsEnv.m

2016-09-20 Thread Anton Tarasov
Hi Sergey, Sure, I’ll send a request. Anton. > On 19 Sep 2016, at 22:51, Sergey Bylokhov wrote: > > Hi, Anton. > Should we backport it to jdk8u? > > On 20.05.16 12:12, Anton Tarasov wrote: >> Thank you! >> >> Anton. >> >>> On 18 May 2016, at 18:54, Alexander Zvegintsev >>> wrote: >>> >>>

Re: [8u/9] Review request 8156116 : [macosx] two JNI locals to delete in AWTWindow.m, CGraphicsEnv.m

2016-09-19 Thread Sergey Bylokhov
Hi, Anton. Should we backport it to jdk8u? On 20.05.16 12:12, Anton Tarasov wrote: Thank you! Anton. On 18 May 2016, at 18:54, Alexander Zvegintsev wrote: Looks fine to me too. Thanks, Alexander. On 05/17/2016 06:36 PM, Anton Tarasov wrote: Would anyone else please review the fix? Tha

Re: [8u/9] Review request 8156116 : [macosx] two JNI locals to delete in AWTWindow.m, CGraphicsEnv.m

2016-05-20 Thread Anton Tarasov
Thank you! Anton. > On 18 May 2016, at 18:54, Alexander Zvegintsev > wrote: > > Looks fine to me too. > > Thanks, > > Alexander. > > On 05/17/2016 06:36 PM, Anton Tarasov wrote: >> Would anyone else please review the fix? >> >> Thanks >> Anton. >> >> On 5/5/2016 12:45 PM, Sergey Bylokhov

Re: [8u/9] Review request 8156116 : [macosx] two JNI locals to delete in AWTWindow.m, CGraphicsEnv.m

2016-05-18 Thread Alexander Zvegintsev
Looks fine to me too. Thanks, Alexander. On 05/17/2016 06:36 PM, Anton Tarasov wrote: Would anyone else please review the fix? Thanks Anton. On 5/5/2016 12:45 PM, Sergey Bylokhov wrote: Looks fine. On 05.05.16 10:50, Anton Tarasov wrote: Hello, Please, review a simple fix: webrev: http: