Re: [PATCH 0/4] MR4852: Add support from AudioPathConfig

[Rémi Bernon (＠rbernon)]

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>".