[PATCH] wineqtdecoder: Fix macos compile error

Alistair Leslie-Hughes leslie_alistair at hotmail.com
Sat Dec 7 22:04:18 CST 2019


Hi Mathew,

On 8/12/19 2:22 pm, Mathew Hodson wrote:
> On Sat, Dec 7, 2019 at 1:03 AM Alistair Leslie-Hughes
> <leslie_alistair at hotmail.com> wrote:
>>
>> Regression of 498179b4482026091bf7376c0d2ac9a036e7ca0e
>>
>> Signed-off-by: Alistair Leslie-Hughes <leslie_alistair at hotmail.com>
>> ---
>>   dlls/wineqtdecoder/qtsplitter.c | 10 ++++------
>>   1 file changed, 4 insertions(+), 6 deletions(-)
>>
>> diff --git a/dlls/wineqtdecoder/qtsplitter.c b/dlls/wineqtdecoder/qtsplitter.c
>> index 5b1e82ebc9b..956dd0a7d0d 100644
>> --- a/dlls/wineqtdecoder/qtsplitter.c
>> +++ b/dlls/wineqtdecoder/qtsplitter.c
>> @@ -285,7 +285,7 @@ static HRESULT qt_splitter_start_stream(struct strmbase_filter *iface, REFERENCE
>>       QTSplitter *filter = impl_from_strmbase_filter(iface);
>>       HRESULT hr = VFW_E_NOT_CONNECTED, pin_hr;
>>
>> -    EnterCriticalSection(&This->csReceive);
>> +    EnterCriticalSection(&filter->csReceive);
> 
> Wouldn't the original commit have failed to compile on all platforms?
The macos build isn't part of the normal build verification process, so 
sometimes things are missed.  This isn't the first, nor will be that 
last ;)

> 
> Where is the This variable defined?
> 

The code was changed to use filter instead of This.

Regards
Alistair.


More information about the wine-devel mailing list