Hi Jim,
I have updated the fix according to your comment:
http://cr.openjdk.java.net/~bae/7002627/webrev/
<http://cr.openjdk.java.net/%7Ebae/7002627/webrev/>
Thanks,
Andrew
On 22.12.2010 0:09, Jim Graham wrote:
Hi Andrew,
Do you really need the "= NULL" on the declarations? They are
initialized on the following line, that should be good enough for any
compiler or lint processing.
Other than that, the new fix looks good...
...jim
On 12/21/2010 2:57 AM, Steve Poole wrote:
Hi Andrew - please feel free to change the patch :-)
Thanks
Steve
From: Andrew Brygin<[email protected]>
To: Steve Poole/UK/i...@ibmgb
Cc: 2d-dev<[email protected]>
Date: 21/12/2010 10:42
Subject: Re: [OpenJDK 2D-Dev]<AWT Dev> 7002627 : JNI Critical Arrays
should be released with the original (unmodified) pointer
Hello Steve,
the fix looks reasonable. However, pMask declaration on line 134
causes a compiler warning:
"../../../src/share/native/sun/awt/../java2d/pipe/BufferedMaskBlit.c",
line 134: warning: declaration can not follow a statement
Would you mind if I modify your fix a bit in order to avoid this
warning?
Please take a look at webrev:
http://cr.openjdk.java.net/~bae/7002627/webrev/
Thanks,
Andrew
On 12/21/2010 12:23 PM, Steve Poole wrote:
Thanks Anthony.
Regards
Steve Poole
From: Anthony Petrov<[email protected]>
To: Steve Poole/UK/i...@ibmgb
Cc: [email protected]
Date: 21/12/2010 09:01
Subject: Re: [OpenJDK 2D-Dev]<AWT Dev> 7002627 : JNI Critical
Arrays
should be released with the original (unmodified) pointer
Sent by: [email protected]
I'm adding the patch attached to the original message on the awt-dev@
list.
--
best regards,
Anthony
On 12/20/2010 3:57 PM, Anthony Petrov wrote:
Hi Steve,
This is a 2D issue, and as such I'm CC'ing 2d-dev@ and BCC'ing
awt-...@.
--
best regards,
Anthony
On 12/20/2010 11:04 AM, Steve Poole wrote:
Hi all - please find attached a patch for your consideration. I've
build
and tested the change on Linux and Solaris at head (which is to say
I've
run the automatic jtreg tests ) and the change doesn't seem to have
broken
anything. Its fairly trivial anyway.
Regards
Steve Poole
(See attached file: 7002627.export)
[attachment "7002627.export" deleted by Steve Poole/UK/IBM]
