17 Jan
2024
17 Jan
'24
8:19 a.m.
Rémi Bernon (@rbernon) commented about dlls/dmime/audiopath.c:
struct audio_path_config *This = impl_from_IUnknown(unk); ULONG ref = InterlockedDecrement(&This->ref);
- if (!ref) + TRACE("(%p): ref=%ld\n", This, ref); + + if (!ref) { + struct audio_path_port_config *i, *tmp;
```suggestion:-0+0 struct audio_path_port_config *config, *next; ``` We prefer longer names in general except for loop integer indices. The safe pointer is also more consistently named "next" or "next_<object>". -- https://gitlab.winehq.org/wine/wine/-/merge_requests/4852#note_57675