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:
> -
>
> +#ifdef WORDS_BIGENDIAN
> +#define BTH_READ_UINT16( s ) (s)
> +#define BTH_READ_UINT32( s ) (s)
> +#define BTH_READ_UINT64( s ) (s)
> +#else
> +#define BTH_READ_UINT16( s ) RtlUshortByteSwap( *(USHORT *)(s) )
> +#define BTH_READ_UINT32( s ) RtlUlongByteSwap( *(ULONG *)(s) )
> +#define BTH_READ_UINT64( s ) RtlUlonglongByteSwap( *(ULONGLONG *)(s) )
> +#endif
> -
>
> +#define BTH_READ_UINT128( s, v )                                                                   \
> - do                                                                                             \
> - {                                                                                              \
> -
>    (v)->HighPart = BTH_READ_UINT64( (s) );                                                    \
> -
>    (v)->LowPart = BTH_READ_UINT64( (s + 8) );                                                 \

```suggestion:-0+0
        (v)->LowPart = BTH_READ_UINT64( ((s) + 8) );                                                 \
```

Then what about a function helper instead?