[PATCH v2 3/3] mmdevapi: Use a struct list to keep device list.

Rémi Bernon rbernon at codeweavers.com
Thu Jun 3 09:33:04 CDT 2021


On 6/3/21 3:54 PM, Marvin wrote:
> Hi,
> 
> While running your changed tests, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> 
> Full results can be found at:
> https://testbot.winehq.org/JobDetails.pl?Key=91753
> 
> Your paranoid android.
> 
> 
> === debiant2 (32 bit report) ===
> 
> mmdevapi:
> mmdevenum.c:405: Test failed: Invalid reference count after incrementing: 2
> mmdevenum.c:54: Test failed: Reference count on parent is 1
> 
> === debiant2 (32 bit Chinese:China report) ===
> 
> mmdevapi:
> mmdevenum.c:405: Test failed: Invalid reference count after incrementing: 2
> mmdevenum.c:54: Test failed: Reference count on parent is 1
> 
> === debiant2 (32 bit WoW report) ===
> 
> mmdevapi:
> mmdevenum.c:405: Test failed: Invalid reference count after incrementing: 2
> mmdevenum.c:54: Test failed: Reference count on parent is 1
> 
> === debiant2 (64 bit WoW report) ===
> 
> mmdevapi:
> mmdevenum.c:405: Test failed: Invalid reference count after incrementing: 2
> mmdevenum.c:54: Test failed: Reference count on parent is 1
> 

Would it be alright to remove these tests? Is it important to keep the 
refcounting for a static COM object?
-- 
Rémi Bernon <rbernon at codeweavers.com>



More information about the wine-devel mailing list