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

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/bluetoothapis/sdp.c:
> -
>        [SDP_SIZEDESC_16_BYTES] = SDP_ST_INT128,
> -
>    },
> - [SDP_TYPE_UUID] =
> -
>    {
> -
>        [SDP_SIZEDESC_2_BYTES] = SDP_ST_UUID16,
> -
>        [SDP_SIZEDESC_4_BYTES] = SDP_ST_UUID32,
> -
>        [SDP_SIZEDESC_16_BYTES] = SDP_ST_UUID128,
> -
>    },
>
> +};
> -
>
> +static BOOL sdp_read_specific_type( BYTE *stream, ULONG stream_size, SDP_SPECIFICTYPE st,
> -
>                                SDP_ELEMENT_DATA *data, SIZE_T *read )
>
> +{
> - switch (st)
> - {
> - case SDP_ST_UINT8:

Same thing wrt consistency, please chose a way to indent cases within a switch, and stick to it.

I personally prefer cases on the same level as the switch like here, but consistency is what matters.