On Wed Apr 12 05:38:56 2023 +0000, Yeshun Ye wrote:
> I'm sorry I tried again, but still failed. I just know that the
> interface should not add 'implicit_handle(handle_t IXXX_HANDLE)' after
> 'uuid(IXXX_UUID)'.
> Can you help me add a simple interface in the testcases and tell me how
> to call the interface function? Otherwise, all I can do is give up.
Could you be more specific about your question? You can paste your code here.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2305#note_29687
On Wed Apr 12 05:47:11 2023 +0000, Huw Davies wrote:
> There's too much going on in this commit / MR. The changes to the
> drivers seem to be addressing a bug rather than implementing the
> Endpoint Volume API.
> Also, note that the values passed to the non-`Scalar` API are in
> decibels, the `Scalar` API appears to be somewhere between linear and
> logrithmic, and the `ISimpleAudioVolume` values are linear.
> Does your app call the `Scalar` versions of the API? If so, a good
> start would be to write some tests to figure out the mapping between
> `Scalar` and non-`Scalar` by e.g. setting using one and retreiving that
> value using the other.
Yes, I'm fixing a bug of an application, not implementing the set of Endpoint Volume API.
Do I really need to write some tests?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2565#note_29674
On Fri Apr 7 10:56:17 2023 +0000, Huw Davies wrote:
> Did these failures get addressed? I don't see testbot runs for the
> later versions of the commits in this MR.
No, I think the function will never return 'MIXERR_INVALCONTROL', so I just run the test again without any change, and it passed. This failures maybe caused by a bug of testbot.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2545#note_29672
On Wed Apr 12 05:38:56 2023 +0000, Zhiyi Zhang wrote:
> Hi Yeshun, you don't need to regularly rebase the MR on top of the
> latest master. Because this area of code is largely undocumented, to
> help this MR gets in, you need to add a test, even if it might not be
> easy. I think you can use some of the example codes in NCALRPC_example. Thanks.
I'm sorry I tried again, but still failed. I just know that the interface should not add 'implicit_handle(handle_t IXXX_HANDLE)' after 'uuid(IXXX_UUID)'.
Can you help me add a simple interface in the testcases and tell me how to call the interface function? Otherwise, all I can do is give up.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2305#note_29671
* Fix InternetGetConnectedStateEx() parameter checking.
* InternetGetConnectedStateExA() must always null-terminate the state string.
* Dump the state string if it is not as expected.
* Remove a couple of redundant InternetGetConnectedStateEx*() tests.
* Avoid an unnecessary lstrlenW() call in internet.c.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/2634
Needed for mingw Firefox build.
The WIDL error points to the wrong function, specifically the one after it.
include/windows.ui.composition.interop.idl:35:63: error: parameter 'swapchain' of function 'CreateCompositionSurfaceForHandle' cannot derive from void *
HRESULT CreateCompositionSurfaceForSwapChain([in] IUnknown *swapchain, [out, retval] ICompositionSurface **result);
^
make[1]: *** [Makefile:163749: include/windows.ui.composition.interop.h] Error 1
--
v3: include: Add windows.ui.composition.interop.idl file.
widl: Add support for WinRT HANDLE parameter type.
https://gitlab.winehq.org/wine/wine/-/merge_requests/2620