Re: Request for approval of 7162476 sync/merge into OpenJDK6

2012-09-03 Thread Andrew Hughes
- Original Message - > Approved. > > -kto > > On Aug 31, 2012, at 11:04 AM, Abhijit Saha wrote: > > > With Oracle security alert [1] went live yesterday, we've to sync > > one fix # 7162476 in OpenJDK6. > > > > > > Here's the webrev/changes for this merge/sync. > >http://cr.openjdk

Re: Request for approval of 7162476 sync/merge into OpenJDK6

2012-09-03 Thread Andrew Hughes
- Original Message - > > Ah ok. Can you point me to where that recent change was reviewed? > > It was not a public alias because 7u7 is security release. > But this isn't a security fix, is it? > Thanks, > SAM > > On 03.09.2012 16:45, Andrew Hughes wrote: > > > > > > - Original Me

Re: Request for approval of 7162476 sync/merge into OpenJDK6

2012-09-03 Thread Andrew Hughes
- Original Message - > > http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/78e01a6ca8d3 > > > > is a completely different changeset (but with the same bug ID) > > which > reverts changes in > > S7162476. The bug& original change explain why the empty > > BoolEditor > > and IntEditor

Re: Request for approval of 7162476 sync/merge into OpenJDK6

2012-09-03 Thread Andrew Hughes
- Original Message - > >> Presumably we also want the consequent regression fix: > >> http://hg.openjdk.java.net/jdk7u/jdk7u/jdk/rev/78e01a6ca8d3 > > > As per Eng, this fix is only for 7ux and > > do we see the same issue with 6-open ? > > Sergei, please comment. > > The 7182135 issue