From: "Claire (@ClearlyClaire)" <wine@gitlab.winehq.org>
Subject: Re: [PATCH v2 0/1] MR338: mmdevapi: Override product strings for DualSense controller audio devices
Message-Id: <wine-wine-mr338-v2-note3048@gitlab-mail-bridge>
Date: Thu, 30 Jun 2022 13:51:42 +0000
In-Reply-To: <wine-wine-mr338-v2@gitlab-mail-bridge>
References: <merge_request_415@gitlab.winehq.org> <wine-wine-mr338-v2@gitlab-mail-bridge>

On Thu Jun 30 13:27:07 2022 +0000, Andrew Eikum wrote:
> This isn't a blocker, but it feels a little silly to get the value in
> `set_driver_prop_value` and then get it again here. You could change the
> `set_driver_prop_value` API to (optionally?) pass back the value it gets
> to the caller, who would then free it.
Hm no matter what I try it feels a bit awkward.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/338#note_3048