RFR: 8327460: Compile tests with the same visibility rules as product code [v2]

[Magnus Ihse Bursie]

On Fri, 8 Mar 2024 18:12:40 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update line number for dereference_null  in TestDwarf
>
> test/jdk/java/foreign/CallGeneratorHelper.java line 216:
> 
>> 214:         if (header) {
>> 215:             System.out.println(
>> 216:                 "#include \"export.h\"\n"
> 
> We don't generate these header files any more, so the changes to this file are not really needed.

I still wouldn't like to keep the bad hard-coded defines. Are you okay with me pushing these changes, and then you can remove the parts of the test that are not actually used anymore? If the code is not used it should not matter much to you either way.

(I mean I could back out these changes, but then we'd have the bad code in place while waiting for you to remove it, putting pressure on you to actually remove it.)

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18135#discussion_r1521503942