<AWT Dev> [OpenJDK 2D-Dev] Request for Review: Remove import of empty JNI header files
philip.race at oracle.com
Wed Apr 25 10:33:54 PDT 2012
Looks fine. If its not a pain, 2d forest would probably be the most
appropriate as 3/4 are 2D ..
On 4/25/2012 8:27 AM, Chris Hegarty wrote:
> [ cc'ing awt-dev & 2d-dev ]
> The change looks fine Magnus, though it may be best to push through
> the awt or 2d forest. Members of these groups, cc'ed, are in a better
> position to comment on this.
> Oh, just to clarify, I agree and approve this change (as much as my
> approval counts ;-) ). Just need to determine the best route for
> On 25/04/2012 14:44, Magnus Ihse Bursie wrote:
>> There are a few files native files in AWT and Java2d, which include
>> generated JNI header files, but the corresponding header files are
>> empty, since the class contains neither native methods nor constants.
>> All of them have probably been useful at one point in time, but then the
>> Java code changed and the include became irrelevant, but nobody noticed.
>> (Some of them contains JDK1.0-style, pre-JNI native access...)
>> The trouble is, in the new build system, we only generate JNI header
>> files when they actually contains something useful! So these files don't
>> get generated. But then the compiler complains when building the native
>> library, since the #include statements points to non-existent files.
More information about the awt-dev