Re: Request for review: 7027045

2013-08-12 Thread Konstantin Perikov
Yes, sure you can use my current gmail Thanks. 2013/8/12 Artem Ananiev > Hi, Konstantin, > > the new version looks fine. > > As Sergey wrote in another email, he will help you with pushing this fix > to the workspace. No further actions are required from your side. Thank you > for help in maki

Re: Request for review: 7027045

2013-08-12 Thread Artem Ananiev
Hi, Konstantin, the new version looks fine. As Sergey wrote in another email, he will help you with pushing this fix to the workspace. No further actions are required from your side. Thank you for help in making Java better :) We usually specify contributor's email in the "Contributed-by" fi

Re: [8] Review request for 7191018: Manual test closed/java/awt/JAWT causes JVM to crash starting from JDK 5

2013-08-12 Thread Anton Litvinov
Hello, Could you please review the already applied request which is presented below. Thank you, Anton On 8/6/2013 9:04 PM, Anton Litvinov wrote: Hello, Could you please review the following fix for a bug from AWT Native Interface area. Bug: http://bugs.sun.com/view_bug.do?bug_id=7191018

Re: Request for review: 7027045

2013-08-12 Thread Konstantin Perikov
So, it will ready to be processed further? Konstantin. 2013/8/12 Sergey Bylokhov > Hi, Konstantin. > Fix looks fine to me. > Thanks! > > On 09.08.2013 14:01, Konstantin Perikov wrote: > > Hi, AWT team > > I update fix. Short summary: > > >- Javadoc fixed from /* to /** for two serializab

Re: Request for review: 7027045

2013-08-12 Thread Sergey Bylokhov
Hi, Konstantin. Fix looks fine to me. Thanks! On 09.08.2013 14:01, Konstantin Perikov wrote: Hi, AWT team I update fix. Short summary: * Javadoc fixed from /* to /** for two serializable fields * Hint about synchronization still exists * Also, I change to {@code}, but left some of them

Re: Request for review: 7027045

2013-08-12 Thread Konstantin Perikov
Hi, AWT team. Any updates on review of my fix? Konstantin. 2013/8/9 Konstantin Perikov > Hi, AWT team > > I update fix. Short summary: > > >- Javadoc fixed from /* to /** for two serializable fields >- Hint about synchronization still exists >- Also, I change to {@code}, but lef