Question about jdwpTransport.h

[Chris Plummer]

It sure looks like a bug. I guess no one has ever used this C++ 
interface, or even included this header file from a C++ compilation. The 
debug agent itself is written in C and calls the C version of this API, 
which is located earlier in the file. So this header file does compile 
from C, but the syntax error you've pointed out is within a #ifdef 
__cplusplus, so is not compiled by the debug agent.

The following link will help you get started with making a contribution:

https://wiki.openjdk.org/display/Adoption/New+Contributor

Thanks,

Chris

On 10/18/22 8:48 AM, 小破 wrote:
> Firstly, this is my first-time sending an email to OpenJDK Community. 
> If I chose a wrong list name in this mailing list 
> <https://mail.openjdk.org/mailman/listinfo>, please forgive me and 
> tell me which email address I should send to.
>
> When I read the JDK source code, I found some codes a little special 
> in the jdwpTransport.h. From line 265 to line 267 (the code link: 
> jdwpTransport.h#L265-L267 
> <https://github.com/openjdk/jdk/blob/master/src/jdk.jdwp.agent/share/native/include/jdwpTransport.h#L265-L267>): 
>
> ```
> jdwpTransportError SetTransportConfiguration(jdwpTransportEnv* env,
> returnfunctions->SetTransportConfiguration(this, config);
> }
> ```
> It seems that there is a syntax error. When I try to compile this 
> file, compilation errors do occur. So, I want to know whether it is a 
> bug or not. If it is, I want to fix it. And then it will be my first 
> time trying to contribute to OpenJDK, I need someone to tell me the 
> process of contributing code to OpenJDK.
>
> Thanks and regards.
>
> ------------------------------------------------------------------------
> 	
> 阿破
> 1052156701 at qq.com
>
> <https://wx.mail.qq.com/home/index?t=readmail_businesscard_midpage&nocheck=true&name=%E9%98%BF%E7%A0%B4&icon=http%3A%2F%2Fthirdqq.qlogo.cn%2Fg%3Fb%3Dsdk%26k%3DuRa3wVPibCtiaxMGbfjYCibaw%26s%3D100%26t%3D1556479581%3Frand%3D1636189913&mail=1052156701%40qq.com&code=>