June 2024 - Wine-GitLab - list.winehq.org

Re: [PATCH v2 0/1] MR5771: wintypes: Report UniversalApiContract version 7 as present.
by Vijay Kiran Kamuju (＠infyquest) June 29, 2024

June 29, 2024

On Sat Jun 29 20:07:51 2024 +0000, Loïc Rebmeister wrote: > @infyquest I'm happy you tried to make a fix, I'm happy you have > interest in wine. > > I dont need you to teach me code, Please don't make it personal. > This ain't personal, if someone want to make a fix, they need at least > to understand the code they send, this is the same with everyone. > I know you wanted to do good, but this doesn't change what this code > effectively does. I am contributing to wine for more than 10 years. I admit that I might be wrong in understanding the code. Please try to communicate and present in a decent way as it will rub the people in a wrong way. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5771#note_74757

1 0

Re: [PATCH v2 0/1] MR5771: wintypes: Report UniversalApiContract version 7 as present.
by Loïc Rebmeister June 29, 2024

June 29, 2024

@infyquest I'm happy you tried to make a fix, I'm happy you have interest in wine. > I dont need you to teach me code, Please don't make it personal. This ain't personal, if someone want to make a fix, they need at least to understand the code they send, this is the same with everyone. I know you wanted to do good, but this doesn't change what this code effectively does. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5771#note_74756

1 0

[PATCH v5 0/1] MR4418: cmd: Delay variable substitution until after the if clause is parsed.
by Alex Henrie (＠alexhenrie) June 29, 2024

June 29, 2024

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55859 -- v5: cmd: Delay variable substitution until after the if clause is parsed. https://gitlab.winehq.org/wine/wine/-/merge_requests/4418

3 4

[PATCH 0/1] MR5956: netapi32: add NetGetAadJoinInformation stub
by Austin English (＠austin987) June 29, 2024

June 29, 2024

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56617 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5956

3 2

Re: [PATCH v2 0/1] MR5771: wintypes: Report UniversalApiContract version 7 as present.
by Vijay Kiran Kamuju (＠infyquest) June 29, 2024

June 29, 2024

I am closing this merge request and wait for the coding teacher to get with a fix. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5771#note_74752

1 0

Re: [PATCH v2 0/1] MR5771: wintypes: Report UniversalApiContract version 7 as present. - closed
by Vijay Kiran Kamuju (＠infyquest) June 29, 2024

June 29, 2024

This merge request was closed by Vijay Kiran Kamuju. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5771

1 0

Re: [PATCH v2 0/1] MR5771: wintypes: Report UniversalApiContract version 7 as present.
by Vijay Kiran Kamuju (＠infyquest) June 29, 2024

June 29, 2024

Please upload the code to the bug and test it. This was an attempt to fix the bug based on observations. There can be issues, with the code. I dont need you to teach me code, Please don't make it personal. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5771#note_74751

1 0

Re: [PATCH v2 0/1] MR5771: wintypes: Report UniversalApiContract version 7 as present.
by Loïc Rebmeister June 29, 2024

June 29, 2024

Your pull request effectively make the code worse, as `{ L"Windows.Foundation.UniversalApiContract", 10, },` means `10` or lower. So `{ L"Windows.Foundation.UniversalApiContract", 7, },` was already included by `{ L"Windows.Foundation.UniversalApiContract", 10, },` But due to how `is_api_contract_present` work, it only take the first element with a name into consideration. This is a simplified pseudo code with comments of `is_api_contract_present` to show the loop stop at first name match: ```java boolean is_api_contract_present(String name,int version) { for (present_contract : present_contracts) // List though all entries if (present_contract.name == name)) // Check is name is equal return version <= present_contract.max_major; // Check if version is lower than max_major, and return return false; // No match found, return that we didn't found anything } ``` Note: the `return` keyword stop code execution, and return the value. (I'm assuming you didn't understood the code you sent there, as I'm pretty sure if you understood that code before sending it you wouldn't have opened this MR) Further more, `Windows.Foundation.UniversalApiContract` is a dotnet 5 feature. (Source: https://github.com/madewokherd/wine-mono/issues/151#issuecomment-1232824011) Wine (and Wine-mono) currently only support up to dotnet 4 on a clean install. If you want I can teach you code for free so you can actually read and understand the code before you are sending it to wine upstream. In the meantime I'll see what I can do to try to get the issue fixed. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5771#note_74750

1 0

[PATCH 0/3] MR5953: include: Add symbols for DirectXShaderCompiler project
by Biswapriyo Nath (＠Biswa96) June 28, 2024

June 28, 2024

Required for https://github.com/microsoft/DirectXShaderCompiler.git -- https://gitlab.winehq.org/wine/wine/-/merge_requests/5953

3 5

[PATCH v9 0/2] MR5761: comclt32: Allow ctrl-tab and tab hotkeys for propsheets
by Fabian Maurer (＠DarkShadow44) June 28, 2024

June 28, 2024

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56103 -- v9: comclt32: Allow hotkeys for propsheet navigation. comctl32/tests: Add test for propsheet hotkey navigation. https://gitlab.winehq.org/wine/wine/-/merge_requests/5761

4 11