compatibility with WIN32_LEAN_AND_MEAN about apriltag HOT 8 OPEN

What's the advantage of using #define WIN32_LEAN_AND_MEAN? According to https://learn.microsoft.com/en-us/windows/win32/winprog/using-the-windows-headers:

Define WIN32_LEAN_AND_MEAN to exclude APIs such as Cryptography, DDE, RPC, Shell, and Windows Sockets.

So it makes sense that some included headers would then be missing.

from apriltag.

Excluding rarely used headers reduces namespace pollution and improves build time. If one want to use it this library should not stand in the way if possible. Instead of including a very general header it should include exactly what it needs.

Here is the link to the fix that we use: modriplanetdoo@45a0a70

from apriltag.

With the diff I better understand what you aim for. I initially understood that you want to use the windows.h header with #define WIN32_LEAN_AND_MEAN and then additionally add the winsock.h header, which would be counterintuitive in my view. Feel free to send a PR with this patch. Also add the required compile flags to the CI in order to make sure that this will not break in future.

Is timeval only directly defined in winsock.h? Is there no specific "time" header that includes this struct?

from apriltag.

As specified in documentation timeval is defined in winsock.h. It it also recommended to use Winsock2.h which is recommended version of this header.

https://learn.microsoft.com/en-us/windows/win32/api/winsock/ns-winsock-timeval
Header winsock.h (include Winsock2.h)

I am not aware of any other time-related header.

from apriltag.

Why is struct timeval (still) used at all might be the question... instead of a plain, standard C or C++ way of dealing with "time"?

from apriltag.

Why is struct timeval (still) used at all might be the question... instead of a plain, standard C or C++ way of dealing with "time"?

Do you know of a standard C way of dealing with time at microsecond resolution?

from apriltag.

A quick pointer could be "https://stackoverflow.com/a/67731965" (https://stackoverflow.com/questions/5833094/get-a-timestamp-in-c-in-microseconds/67731965#67731965).

Haven't grepped AprilTag for where timeval is used... why is a timestamp needed to process AprilTags ;-) ?

from apriltag.

A quick pointer could be "https://stackoverflow.com/a/67731965" (https://stackoverflow.com/questions/5833094/get-a-timestamp-in-c-in-microseconds/67731965#67731965).

The response you linked recommends not using timespec_get. So in the end, you are left with the native APIs. The question and the answer are using the POSIX function gettimeofday which is also the one used here.

Haven't grepped AprilTag for where timeval is used... why is a timestamp needed to process AprilTags ;-) ?

The time is used for profiling.

from apriltag.