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. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6402#note_82806