C99 API: #include <bx/platform.h> only if it's needed

[jdryg]

As far as I understand bx/platform.h is needed inside bgfx/c99/bgfx.h for the BX_PLATFORM_xxx macro which in turn is used to define BGFX_SYMBOL_EXPORT and BGFX_SYMBOL_IMPORT.

Both of those macros are used only if at least one of BGFX_SHARED_LIB_BUILD or BGFX_SHARED_LIB_USE is defined.

In case the bgfx API is exposed via another method (e.g. in a plugin system the bgfx_interface_vtbl_t pointer might have been provided to the plugin by the host), there is no need to depend on bx just for this header.

Is it possible to hide bx/platform.h in those cases?

E.g.

#if BGFX_SHARED_LIB_BUILD || BGFX_SHARED_LIB_USE
#include <bx/platform.h>

#if BX_PLATFORM_WINDOWS || BX_PLATFORM_WINRT
#   define BGFX_SYMBOL_EXPORT __declspec(dllexport)
#   define BGFX_SYMBOL_IMPORT __declspec(dllimport)
#else
#   define BGFX_SYMBOL_EXPORT __attribute__((visibility("default")))
#   define BGFX_SYMBOL_IMPORT
#endif // BX_PLATFORM_WINDOWS
#endif

It's not a big deal, but I thought of asking.