<div dir="ltr">Not to make too big a deal of it, but the idea was that if you went out of your way to define DRM_FOURCC_STANDALONE in your code base, that you would also go through the pain of removing drm.h includes elsewhere. It's too annoying of an implication to document/communicate, so I'm happier with the other DRM_FOURCC_STANDALONE solution that pulls the basic types into a common header.<div><br></div><div>Thanks,</div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 7, 2020 at 1:49 AM Simon Ser <<a href="mailto:contact@emersion.fr">contact@emersion.fr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Monday, December 7th, 2020 at 9:57 AM, James Park <<a href="mailto:james.park@lagfreegames.com" target="_blank">james.park@lagfreegames.com</a>> wrote:<br>
<br>
> I could adjust the block to look like this:<br>
><br>
> #ifdef DRM_FOURCC_STANDALONE<br>
> #if defined(__linux__)<br>
> #include <linux/types.h><br>
> #else<br>
> #include <stdint.h><br>
> typedef uint32_t __u32;<br>
> typedef uint64_t __u64;<br>
> #endif<br>
> #else<br>
> #include "drm.h"<br>
> #endif<br>
<br>
This approach still breaks on BSDs when DRM_FOURCC_STANDALONE is defined and<br>
drm.h is included afterwards.<br>
</blockquote></div>