|
From: James T. <ja...@fl...> - 2022-08-24 08:00:43
|
> On 23 Aug 2022, at 15:57, James Hogan via Flightgear-devel <fli...@li...> wrote: > > Sorry about that. Does the following patch fix it for you? > Feels a bit wrong that you have to magically know what to include before > openxr_platform.h in the presence of future extensions, but the > xr_dependencies.h header in the openxr sdk appears to be internal to the > loader. This does feel a bit weird to me : I’ve never seen any code before, need to manually include unknwn.h before (I think it’s one of those things you are meant to get via a specified top-level include, not pull in yourself) So, maybe we consider this a work-around, anyway : it’s not a terrible fix, just … odd … so at least a comment stating why it’s needed? Kind regards, James |