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.