Re: [PATCH 0/4] MR7707: msvcp140: Implement codecvt<char16_t, char> conversions.

[Piotr Caban (＠piotr)]

Piotr Caban (@piotr) commented about dlls/msvcp140/tests/msvcp140.c:
>  p_setlocale(LC_ALL, "C");
>
> }
>
> +void test_codecvt_char16(void)
> +{
> - char buffer[256];
> - codecvt_char16 *this, *this2;
> -
> - this = (codecvt_char16 *)buffer;

Is there any reason for using `char[256]` buffer here instead of `codecvt_char16`? It's usually not very useful to check structure size this way since some fields may be unused in constructor.