Re: [PATCH v2 0/5] MR8037: widl: Initial support for generating Windows Runtime metadata.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about tools/widl/metadata.c:
> -
> - This library is distributed in the hope that it will be useful,
> -
> - but WITHOUT ANY WARRANTY; without even the implied warranty of
> -
> - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> -
> - Lesser General Public License for more details.
> -
> -
> -
> - You should have received a copy of the GNU Lesser General Public
> -
> - License along with this library; if not, write to the Free Software
> -
> - Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
> - */
> -
>
> +#include "config.h"
> +#include <stdarg.h>
> +#include <stdio.h>
> -
>
> +#include "windef.h"
> +#include "winnt.h"

I'm wondering if it's alright to use Win32 types in `widl`? Other unix-only programs seem to prefer using C builtin types instead (`unsigned int`, `size_t`, `bool` from `stdbool`, etc). I don't have a strong opinion but using Win32 types will more tightly couple the code with Wine/Win32 headers.