<div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Alexey,<div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 26, 2019 at 2:36 PM Alexey Ivanov <<a href="mailto:alexey.ivanov@oracle.com">alexey.ivanov@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Thomas,<br>
<br>
Looks good, I'm not a reviewer though.<br>
Could I also ask you to push this changeset to jdk/client [1] instead of <br>
jdk/jdk?<br>
<br>
A small question: two prototypes are changed in debug_trace.h but only <br>
one is updated in debug_trace.c. Is it because the functions matching <br>
the second prototype have already been updated with JNICALL?<br>
<br></blockquote><div><br></div><div>Not really sure I understand the question. There is no real 1:1 relationship between my change in .h and .c:</div><div><br></div><div><div>DTRACE_PRINTxx macros call _DTrace_Template() which expands to DTrace_PrintFunction(). First argument is pointer to DTrace_VPrint(), which is a function decorated with JNICALL. Hence the build error - the type of the first argument of DTrace_PrintFunction is DTRACE_PRINT_CALLBACK, which is a fkt pointer *without* JNICALL.</div><div><br></div><div>So one way to fix this was to correct the DTRACE_PRINT_CALLBACK type to be a fkt pointer pointing to a JNICALL decorated function.</div><div><br></div><div>That means that I also needed to fix all functions whose pointers are being passed around as DTRACE_PRINT_CALLBACK. But there only was one I saw, DTrace_PrintStdErr().</div><div><br></div><div>I'll push to jdk/client. Will that be transported to jdk/jdk automatically?</div><div><br></div><div>Cheers, Thomas</div><div><br></div><div><br></div><div> </div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Regards,<br>
Alexey<br>
<br>
[1] <a href="http://hg.openjdk.java.net/jdk/client/" rel="noreferrer" target="_blank">http://hg.openjdk.java.net/jdk/client/</a><br>
<br>
On 25/03/2019 13:09, Thomas Stüfe wrote:<br>
> Hi all,<br>
><br>
> Issue: <a href="https://bugs.openjdk.java.net/browse/JDK-8221405" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8221405</a><br>
> cr: <br>
> <a href="http://cr.openjdk.java.net/~stuefe/webrevs/8221405-windows32-awt-buildfixes/webrev.00/webrev/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~stuefe/webrevs/8221405-windows32-awt-buildfixes/webrev.00/webrev/</a><br>
><br>
> On 32bit windows, awt build did bitrot. Dtrace print callbacks need to <br>
> be declared with __stdcall. No other platform cares.<br>
><br>
> Thanks, Thomas<br>
<br>
</blockquote></div></div></div>