<AWT Dev> [8] Request for review: 8020298 [macosx] Incorrect merge in the lwawt code.

Anthony Petrov anthony.petrov at oracle.com
Fri Jul 12 07:57:56 PDT 2013


Both fixes look fine to me.

--
best regards,
Anthony

On 07/12/2013 06:36 PM, Sergey Bylokhov wrote:
> Hello,
> Please review the fix for jdk 7/8.
> The problem is in incorrect merge between 8009012 and 8003559.
> I remove part(optimization) of the 8009012, because it was merged
> incorrectly and because it require additional testing after cpu merge.
> Also a call to flushBuffers was removed, when the peer is null, because
> it does not make sense in this case and can hang.
>
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8020298
> Webrevs can be found at:
> http://cr.openjdk.java.net/~serb/8020298/webrev.8/
> http://cr.openjdk.java.net/~serb/8020298/webrev.7/
> Webrevs for jdk8 and jdk7 are different but result is the same.
>


More information about the awt-dev mailing list