May 2024 - Wine-gitlab - list.winehq.org

Re: [PATCH v2 0/1] MR5722: dxdiagn: Add bIsD3DDebugRuntime property. - approved
by Alexandre Julliard (＠julliard) 28 May '24

28 May '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5722

1 0

Re: [PATCH v2 0/1] MR5726: dxdiagn: Add AGP properties. - approved
by Alexandre Julliard (＠julliard) 28 May '24

28 May '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5726

1 0

Re: [PATCH v2 0/2] MR5729: Fix SQLSetConnectOptionW prototype - approved
by Alexandre Julliard (＠julliard) 28 May '24

28 May '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5729

1 0

Re: [PATCH v2 0/2] MR5734: mf: Implement audio capture device enumeration in MFEnumDeviceSources().
by Paul Gofman (＠gofman) 28 May '24

28 May '24

On Mon May 27 20:51:49 2024 +0000, Nikolay Sivov wrote: > It's unlikely those device strings need to be constructed in client > library here, I'd expect those id to come from setupapi > (DEVPKEY_Device_InstanceId?), if they are not present in mmdevice > property store already. If application really depends on those > attributes we could add them here as a temporary solution, but if it > doesn't, I'd rather prefer a fixme. Device symlink is required for video > sources I think, but for mmdevapi you'd use an endpoint id. I didn't find a straightforward way to get it from mmdevapi. No, I don't know that anything (that is, the only app I came through) is using that link (although I think it is possible to create mmdevapi device using that path with, e. g., ActivateAudioInterfaceAsync). I will look a bit more and if no obvious way I will remove this field for now. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5734#note_71444

1 0

Re: [PATCH v2 0/2] MR5730: Various Include changes - approved
by Alexandre Julliard (＠julliard) 28 May '24

28 May '24

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5730

1 0

Re: [PATCH v2 0/2] MR5734: mf: Implement audio capture device enumeration in MFEnumDeviceSources().
by Paul Gofman (＠gofman) 28 May '24

28 May '24

On Mon May 27 20:51:50 2024 +0000, Nikolay Sivov wrote: > What does "sac" stand for? Source Audio Capture. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5734#note_71442

1 0

Re: [PATCH v2 0/2] MR5734: mf: Implement audio capture device enumeration in MFEnumDeviceSources().
by Nikolay Sivov (＠nsivov) 28 May '24

28 May '24

Nikolay Sivov (@nsivov) commented about dlls/mf/sac.c: > + > + hr = IMMDevice_OpenPropertyStore(device, STGM_READ, &ps); > + IMMDevice_Release(device); > + if (FAILED(hr)) > + goto done; > + hr = IPropertyStore_GetValue(ps, (const PROPERTYKEY*)&DEVPKEY_Device_FriendlyName, &pv); > + IPropertyStore_Release(ps); > + if (FAILED(hr)) > + goto done; > + if (pv.vt != VT_LPWSTR) > + { > + PropVariantClear(&pv); > + goto done; > + } > + IMFActivate_SetString(sources[i], &MF_DEVSOURCE_ATTRIBUTE_FRIENDLY_NAME, pv.pwszVal); > + PropVariantClear(&pv); I think cascading checks for hr are better here than multiple jumps. And if property store becomes useful for instance id, we could have a helper that checks value types. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5734#note_71440

1 0

Re: [PATCH v2 0/2] MR5734: mf: Implement audio capture device enumeration in MFEnumDeviceSources().
by Nikolay Sivov (＠nsivov) 28 May '24

28 May '24

Nikolay Sivov (@nsivov) commented about dlls/mf/main.c: > - FIXME("%p, %p, %p.\n", attributes, sources, count); > + GUID source_type; > + HRESULT hr; > + > + TRACE("%p, %p, %p.\n", attributes, sources, ret_count); > > - if (!attributes || !sources || !count) > + if (!attributes || !sources || !ret_count) > return E_INVALIDARG; > > - *count = 0; > + if (FAILED(hr = IMFAttributes_GetGUID(attributes, &MF_DEVSOURCE_ATTRIBUTE_SOURCE_TYPE, &source_type))) > + return hr; > > - return S_OK; > + if (IsEqualIID(&source_type, &MF_DEVSOURCE_ATTRIBUTE_SOURCE_TYPE_VIDCAP_GUID)) It makes no difference in practice, but IsEqualGUID() is a better fit. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5734#note_71439

1 0

Re: [PATCH v2 0/2] MR5734: mf: Implement audio capture device enumeration in MFEnumDeviceSources().
by Nikolay Sivov (＠nsivov) 28 May '24

28 May '24

Nikolay Sivov (@nsivov) commented about dlls/mf/main.c: > + HRESULT hr; > + > + TRACE("%p, %p, %p.\n", attributes, sources, ret_count); > > - if (!attributes || !sources || !count) > + if (!attributes || !sources || !ret_count) > return E_INVALIDARG; > > - *count = 0; > + if (FAILED(hr = IMFAttributes_GetGUID(attributes, &MF_DEVSOURCE_ATTRIBUTE_SOURCE_TYPE, &source_type))) > + return hr; > > - return S_OK; > + if (IsEqualIID(&source_type, &MF_DEVSOURCE_ATTRIBUTE_SOURCE_TYPE_VIDCAP_GUID)) > + { > + FIXME("Not implementted for video capture devices.\n"); Typo here. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5734#note_71438

1 0

Re: [PATCH v2 0/2] MR5734: mf: Implement audio capture device enumeration in MFEnumDeviceSources().
by Nikolay Sivov (＠nsivov) 28 May '24

28 May '24

Nikolay Sivov (@nsivov) commented about dlls/mf/Makefile.in: > mf.rc \ > quality.c \ > samplegrabber.c \ > + sac.c \ What does "sac" stand for? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5734#note_71437

1 0