RFR: 8307160: Fix AWT/2D/A11Y to support the permissive- flag on the Microsoft Visual C compiler [v49]
Julian Waters
jwaters at openjdk.org
Mon Apr 1 10:36:54 UTC 2024
On Sun, 21 Jan 2024 19:50:16 GMT, Phil Race <prr at openjdk.org> wrote:
>> Fixed the formatting (at least in the marked cases), but am unsure what you mean by set directly?
>
>> Fixed the formatting (at least in the marked cases), but am unsure what you mean by set directly?
>
> See my comment
> "like in my recoded case above, we no longer need the "pData" var which was there only because that name
> is magically known to the macro."
>
> so skip (and get rid of) the pData var and just set the target var directly
I've switched the marked cases to use direct setting and bypass pData, should I do the unmarked ones too? @prrace
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15096#issuecomment-2024592358
More information about the build-dev
mailing list