Re: [PATCH v14 0/9] MR6621: winebth.sys part 1: Introduce Bluetooth driver.

[Elizabeth Figura (＠zfigura)]

Elizabeth Figura (@zfigura) commented about dlls/winebth.sys/winebth.c:
> - static CRITICAL_SECTION cs;                                                                    \
> - static CRITICAL_SECTION_DEBUG cs##_debug = {                                                   \
> -
>    0,                                                                                         \
> -
>    0,                                                                                         \
> -
>    &( cs ),                                                                                   \
> -
>    { &cs##_debug.ProcessLocksList, &cs##_debug.ProcessLocksList },                            \
> -
>    0,                                                                                         \
> -
>    0,                                                                                         \
> -
>    { (DWORD_PTR)( __FILE__ ": " #cs ) } };                                                    \
> - static CRITICAL_SECTION cs = { &cs##_debug, -1, 0, 0, 0, 0 };
> -
>
> +DECLARE_CRITICAL_SECTION( device_list_cs );
> -
>
> +static struct list device_list = LIST_INIT( device_list );
> -
>
> +static winebluetooth_watcher_t watcher;

Why are we storing this on the PE side? There's only one of these; it's not like we need it to underly a PE object.