Fix Windows Build

[HTRamsey]

Fixes a couple of windows issues

Edit: CMake should be updated too (like the compile options) but it shouldn't affect using this as a submodule since MAIN_PROJECT would be false

[bkueng]

There's a minor style issue, otherwise good to merge (https://github.com/PX4/ulog_cpp/actions/runs/10309113259/job/28554831362?pr=10).

[HTRamsey]

So based on this (https://stackoverflow.com/questions/4845198/fatal-error-no-target-architecture-in-visual-studio) windows.h needs to be included first otherwise I get "No Target Architecture". But maybe we just use a different method to define it, like this?

#if defined(__amd64__) || defined(__amd64) || defined(__x86_64__) || defined(__x86_64) || defined(_M_X64) || defined(_M_AMD64)
#define _AMD64_
#elif defined(i386) || defined(__i386) || defined(__i386__) || defined(__i386__) || defined(_M_IX86)
#define _X86_
#elif defined(__arm__) || defined(_M_ARM) || defined(_M_ARMT)
#define _ARM_
#endif

Probably this wouldn't happen when using this as a submodule, just when using the tests directly.

[bkueng]

Ah ok, then let's just ignore it with // NOLINT or // NOLINTBEGIN / // NOLINTEND