Elizabeth Figura (@zfigura) commented about programs/plugplay/main.c:
EnterCriticalSection( &plugplay_cs );
- while (!(entry = list_head( &listener->events ))) + while(!(entry = list_head( &listener->events ))) SleepConditionVariableCS( &listener->cv, &plugplay_cs, INFINITE );
- event = LIST_ENTRY(entry, struct event, entry); - list_remove( &event->entry ); + event = LIST_ENTRY( entry, struct event, entry ); + list_remove(&event->entry);
There's a lot of unnecessary whitespace changes here, and they run contrary to the usual style in Wine. We use a space after if/while/switch, (at least in user32 and plugplay) a space inside function calls and function-like macros (but no space inside if/while/switch conditions). Some other code in this merge request also has inconsistent spacing. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/6315#note_79366