tech-userlevel archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: EV_SET() better C++ compat with alternative implementations



On Sun, Aug 11, 2019 at 07:38:21PM +0200, Kamil Rytarowski wrote:
> It used to break C ones too, but we have introduced a cast to workaround it.
> 
> As we fix it for C ones, I think we should fix it for C++ ones as well.
> 
> 3 solutions:
> 
>  - Keep patching it on caller level for C++
>  - Keep inventing workarounds on callee level for C++
>  - Change the type of udata to void* in our source code.

So, this is when we changed to intptr_t. We also went through a
"int64_t udata" revision at some point, later.


Module Name:    syssrc
Committed By:   jdolecek
Date:           Wed Oct  2 19:09:37 UTC 2002

Modified Files:
        syssrc/sys/sys [kqueue]: event.h

Log Message:
make 'udata' field of struct kevent intptr_t (was 'void *')


To generate a diff of this commit:
cvs rdiff -r1.1.1.1.2.10 -r1.1.1.1.2.11 syssrc/sys/sys/event.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

jdolecek, do you remember why you wanted to switch?


Home | Main Index | Thread Index | Old Index