<AWT Dev> RFR: JDK-7181710: [macosx] jawt_md.h shipped with jdk is outdated

Anthony Petrov anthony.petrov at oracle.com
Thu Jun 6 06:21:38 PDT 2013


Thanks Artem.

David: I can push the fix as soon as you get a release team approval for 
this fix to go to 7u-dev.

--
best regards,
Anthony

On 06/06/13 17:10, Artem Ananiev wrote:
>
> The changes looks fine to me. For JDK8, the fix can be simplified by
> removing X11-specific part of jawt_md.h on Mac, since we're going to
> remove XAWT support on that platform in JDK8 anyway.
>
> Thanks,
>
> Artem
>
> On 6/6/2013 4:40 PM, Anthony Petrov wrote:
>> I've already reviewed this fix, and it looks fine to me. We still need
>> another reviewer. Adding awt-dev at .
>>
>> Anyone please?
>>
>> --
>> best regards,
>> Anthony
>>
>> On 06/05/13 20:52, David DeHaven wrote:
>>>
>>> Proposed fix for getting the correct jawt_md.h header in the JDK. I
>>> reused the XAWT flag to enable the X11 drawing surface struct. It is
>>> disabled by default as just about every application running on Mac
>>> will use the Cocoa interface. Apparently the X11 interface is still
>>> operable on Mac and it's used for verification, so I don't want to
>>> completely remove it just yet. I also changed several #include
>>> statements to use the JDK headers instead of the deprecated JavaVM
>>> framework headers.
>>>
>>> Since I'm not a jdk7u-dev committer, Anthony Petrov has graciously
>>> agreed to sponsor this patch. I will be kicking off a JPRT run when I
>>> can steal another couple minutes, and figure out the magic
>>> incantations needed to include the AWT tests.
>>>
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~ddehaven/7181710/jdk.0/
>>>
>>> Issue:
>>> http://bugs.sun.com/view_bug.do?bug_id=7181710
>>>
>>> -DrD-
>>>


More information about the awt-dev mailing list