Re: [PATCH v15 0/6] MR6402: bluetoothapis: Implement basic SDP methods.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/bluetoothapis/tests/sdp.c:
> -
>    "Expected BluetoothSdpGetElementData to return %d, got %ld.\n", ERROR_INVALID_PARAMETER,
> -
>    ret );
>
> +}
> -
>
> +static void test_BluetoothSdpGetElementData_nil( void )
> +{
> - static struct
> - {
> -
>    BYTE data_elem;
> -
>    DWORD error;
> -
>    SDP_ELEMENT_DATA data;
> - } test_cases[] = {
> -
>    {0b00000000, ERROR_SUCCESS, {.type = SDP_TYPE_NIL, .specificType = SDP_ST_NONE }},
> -
>    {0b00000001, ERROR_INVALID_PARAMETER},
> -
>    {0b00000011, ERROR_INVALID_PARAMETER},
> -
>    {0b00000100, ERROR_INVALID_PARAMETER},

Same thing here, do we need 0b? I have no idea how "safe" it is to use it, I believe we keep MSVC compatibility for anything that is built as PE, does it work there?

IMO would be simpler to avoid the question and not use it.