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:
> - case SDP_TYPE_INT:
> - case SDP_TYPE_UUID:
> -
>    if (size_desc > SDP_SIZEDESC_16_BYTES) return ERROR_INVALID_PARAMETER;
> -
> -
>    st = SDP_BASIC_TYPES[type][size_desc];
> -
>    if (st == SDP_ST_NONE) return ERROR_INVALID_PARAMETER;
> -
> -
>    if (!sdp_read_specific_type( stream, stream_size, st, data, read ))
> -
>        return ERROR_INVALID_PARAMETER;
> -
> -
>    data->type = type;
> -
>    data->specificType = st;
> -
>    break;
> - case SDP_TYPE_BOOLEAN:
> -
>    if (size_desc != SDP_SIZEDESC_1_BYTE) return ERROR_INVALID_PARAMETER;
> -
>    if (stream_size < sizeof(BYTE)) return ERROR_INVALID_PARAMETER;

A bit sad that this one lost its parens.

Fwiw, the "space in paren Wine code style" usually only applies to function decls and calls and macros, with a couple of exceptions "where it looks better". So, sizeof operator doesn't usually have spaces inside.

I don't want to bikeshed about it, but please be consistent.