libsdl-org / sdl12-compat

An SDL-1.2 compatibility layer that uses SDL 2.0 behind the scenes.
Other
194 stars 40 forks source link

Check for undefined TARGET_OS_* #293

Closed jmroot closed 1 year ago

jmroot commented 1 year ago

Prevents some versions of clang from erroring out like this:

error: 'TARGET_OS_TV' is not defined, evaluates to 0 [-Werror,-Wundef-prefix=TARGET_OS_]
icculus commented 1 year ago

We might need this fix in the real SDL headers too.

slouken commented 1 year ago

Yup, I believe we do.

sezero commented 1 year ago

Those aren't actually in real SDL-1.2 headers.

sezero commented 1 year ago

Pushed https://github.com/libsdl-org/sdl12-compat/commit/c1eb2fecf9c5fc05e2efce0e2a135cd7474a4874 as a follow-up.