On 6/28/22 10:10 AM, Ammar Faizi wrote:
> On 6/28/22 10:04 PM, Dylan Yudaken wrote:
>> copy from include/uapi/linux/io_uring.h
>>
>> Signed-off-by: Dylan Yudaken <dylany@xxxxxx>
>> ---
>> src/include/liburing/io_uring.h | 53 ++++++++++++++++++++++++---------
>> 1 file changed, 39 insertions(+), 14 deletions(-)
>>
>> diff --git a/src/include/liburing/io_uring.h b/src/include/liburing/io_uring.h
>> index 2f391c9..1e5bdb3 100644
>> --- a/src/include/liburing/io_uring.h
>> +++ b/src/include/liburing/io_uring.h
>> @@ -10,10 +10,7 @@
>> #include <linux/fs.h>
>> #include <linux/types.h>
>> -
>> -#ifdef __cplusplus
>> -extern "C" {
>> -#endif
>
> Dylan,
>
> That `extern "C"` thing is for C++, we shouldn't omit it.
>
> Or better add that to the kernel tree as well, it won't break
> the kernel because we have a __cplusplus guard here.
>
> Jens what do you think?
It'd be nice to keep them fully in sync. If I recall correctly, the only
differences right now is that clause, and the change to not using a zero
sized array at the end of a struct (which is slated for the kernel too).
--
Jens Axboe
