- Wine-GitLab - list.winehq.org

Re: [PATCH v4 0/1] MR677: kerberos: Fix regression in kerberos_SpAcceptLsaModeContext and kerberos_SpAcceptLsaModeContext
by Konstantin (＠IncubusRK) Aug. 19, 2022

Aug. 19, 2022

On Fri Aug 19 07:19:10 2022 +0000, Konstantin wrote: > @hans, thank you for comment. If I understand correct you are author of > original logic. > I am not sure about changes, your suggestion change existing logic. > Originaly `*mapped_context` and `expiry_to_timestamp( exptime, expiry > );` executed only when status is SEC_E_OK, and `new_context` > modification (independant) internaly in `create_context_handle`. > I try change condition to ` if > (status==SEC_E_OK||status==SEC_I_CONTINUE_NEEDED) ` and test it in our network. Tested the changes, kerberos auth in our client application work fine. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/677#note_6526

1 0

Re: [PATCH v6 0/1] MR672: Draft: Common item dialog fixes
by Ben Cottrell (＠bgc) Aug. 19, 2022

Aug. 19, 2022

On Thu Aug 18 07:09:33 2022 +0000, Zhiyi Zhang wrote: > Do not remove the id property. There should be a id="bitmap:width-depth" > property for the wine's build system to generate the bitmaps from SVGs. > Also, the commit subject is way too long. > And, for the WineHQ bugzilla link. We use the Wine-Bug: tag. OK, id properties have been kept, and have shortened the commit message. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/672#note_6523

1 0

Re: [PATCH v2 0/1] MR677: kerberos: Fix regression in kerberos_SpAcceptLsaModeContext and kerberos_SpAcceptLsaModeContext
by Konstantin (＠IncubusRK) Aug. 19, 2022

Aug. 19, 2022

On Fri Aug 19 06:54:19 2022 +0000, Konstantin wrote: > @hans, thank you for comment. If I understand correct you are author of > original logic. > I am not sure. You sugesstion change existing logic. Originaly > `*mapped_context` and `expiry_to_timestamp( exptime, expiry );` executed > only when status is SEC_E_OK, and `new_context` modification > (independant) internaly in `create_context_handle`. > I can't check side effect of the proposed change. > If you sure that it correct I change condition to ` if > (status==SEC_E_OK||status==SEC_I_CONTINUE_NEEDED) ` What about SEC_I_COMPLETE_NEEDED and SEC_I_COMPLETE_AND_CONTINUE we need also put it to condition? -- https://gitlab.winehq.org/wine/wine/-/merge_requests/677#note_6521

1 0

Re: [PATCH v2 0/1] MR677: kerberos: Fix regression in kerberos_SpAcceptLsaModeContext and kerberos_SpAcceptLsaModeContext
by Konstantin (＠IncubusRK) Aug. 19, 2022

Aug. 19, 2022

On Thu Aug 18 14:47:05 2022 +0000, Hans Leidekker wrote: > As I mentioned in the bug report, the new context is updated when > initialize_context() returns SEC_E_OK or SEC_I_CONTINUE_NEEDED. That > should be reflected here. @hans, thank you for comment. If I understand correct you are author of original logic. I am not sure. You sugesstion change existing logic. Originaly `*mapped_context` and `expiry_to_timestamp( exptime, expiry );` executed only when status is SEC_E_OK, and `new_context` modification (independant) internaly in `create_context_handle`. I can't check side effect of the proposed change. If you sure that it correct I change condition to ` if (status==SEC_E_OK||status==SEC_I_CONTINUE_NEEDED) ` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/677#note_6520

1 0

[PATCH v5 0/1] MR672: Draft: Common item dialog fixes
by Ben Cottrell (＠bgc) Aug. 19, 2022

Aug. 19, 2022

Made changes to comdlg32/itemdlg.c to add missing controls, which are the address bar and the up button. Updated bitmaps in comctl32 and SVG files for history icons. My changes help to resolve this bug: https://bugs.winehq.org/show_bug.cgi?id=50338. -- v5: comdlg32: Add missing controls and their functionality to IFileDialog, or the common item dialog. https://gitlab.winehq.org/wine/wine/-/merge_requests/672

3 3

[PATCH v5 0/1] MR23: shell32: Sanitize Program Manager icon and group names
by Alex Henrie (＠alexhenrie) Aug. 19, 2022

Aug. 19, 2022

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52506 Signed-off-by: Alex Henrie <alexhenrie24(a)gmail.com> -- v5: shell32: Sanitize Program Manager icon and group names. https://gitlab.winehq.org/wine/wine/-/merge_requests/23

2 1

[PATCH v6 0/2] MR628: win32u: Fix the second_name parsed wrong.
by Jiajin Cui (＠jin-king1) Aug. 19, 2022

Aug. 19, 2022

When first parsed to names in english and other non-native languages, after parsing to names in native languages, this will result in second_name not corresponding to names in english. Signed-off-by: Jiajin Cui <cuijiajin(a)uniontech.com> -- v6: win32u: Make the replacement font not the replaced font. https://gitlab.winehq.org/wine/wine/-/merge_requests/628

2 3

Re: [PATCH v2 0/6] MR674: win32u: wow64 fixes and winex11 support - approved
by Alexandre Julliard (＠julliard) Aug. 18, 2022

Aug. 18, 2022

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

1 0

Re: [PATCH v2 0/3] MR659: comctl32/tests: Test that changing focus for edit controls doesn't repaint immediately. - approved
by Alexandre Julliard (＠julliard) Aug. 18, 2022

Aug. 18, 2022

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

1 0

Re: [PATCH v2 0/6] MR660: user32/tests: Link directly to imm32. - approved
by Alexandre Julliard (＠julliard) Aug. 18, 2022

Aug. 18, 2022

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

1 0

Re: [PATCH v2 0/4] MR676: mfreadwrite/writer: Add sample/marker queue for each stream. - approved
by Alexandre Julliard (＠julliard) Aug. 18, 2022

Aug. 18, 2022

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

1 0

Re: [PATCH v2 0/1] MR673: msvcp71: Use MultiByteToWideChar in _Fiopen implemen. - approved
by Alexandre Julliard (＠julliard) Aug. 18, 2022

Aug. 18, 2022

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

1 0

[PATCH 0/2] MR669: gdi32/tests: Add more tests for GetTextMetrics().
by Zhiyi Zhang (＠zhiyi) Aug. 18, 2022

Aug. 18, 2022

Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/669

4 4

[PATCH 0/3] MR659: comctl32/tests: Test that changing focus for edit controls doesn't repaint immediately.
by Zhiyi Zhang (＠zhiyi) Aug. 18, 2022

Aug. 18, 2022

Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/659

3 4

[PATCH 0/1] MR677: kerberos: Fix regression in kerberos_SpAcceptLsaModeContext and kerberos_SpAcceptLsaModeContext
by Konstantin (＠IncubusRK) Aug. 18, 2022

Aug. 18, 2022

https://bugs.winehq.org/show_bug.cgi?id=53472 -- https://gitlab.winehq.org/wine/wine/-/merge_requests/677

3 2

[PATCH 0/6] MR660: user32/tests: Link directly to imm32.
by Zhiyi Zhang (＠zhiyi) Aug. 18, 2022

Aug. 18, 2022

Signed-off-by: Zhiyi Zhang <zzhang(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/660

4 12

[PATCH 0/4] MR676: mfreadwrite/writer: Add sample/marker queue for each stream.
by Nikolay Sivov (＠nsivov) Aug. 18, 2022

Aug. 18, 2022

Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> -- https://gitlab.winehq.org/wine/wine/-/merge_requests/676

2 4

[PATCH 0/6] MR674: win32u: wow64 fixes and winex11 support
by Jacek Caban (＠jacek) Aug. 18, 2022

Aug. 18, 2022

I also created a branch that I use for testing wow64: https://gitlab.winehq.org/jacek/wine/-/commits/wow -- https://gitlab.winehq.org/wine/wine/-/merge_requests/674

4 8

[PATCH v4 0/1] MR672: Draft: Common item dialog fixes
by Ben Cottrell (＠bgc) Aug. 18, 2022

Aug. 18, 2022

Made changes to comdlg32/itemdlg.c to add missing controls, which are the address bar and the up button. Updated bitmaps in comctl32 and SVG files for history icons. My changes help to resolve this bug: https://bugs.winehq.org/show_bug.cgi?id=50338. -- v4: comdlg32: Add missing controls and their functionality to IFileDialog, or the common item dialog. https://gitlab.winehq.org/wine/wine/-/merge_requests/672

3 2

[PATCH v3 0/2] MR628: win32u: Fix the second_name parsed wrong.
by Jiajin Cui (＠jin-king1) Aug. 18, 2022

Aug. 18, 2022

When first parsed to names in english and other non-native languages, after parsing to names in native languages, this will result in second_name not corresponding to names in english. Signed-off-by: Jiajin Cui <cuijiajin(a)uniontech.com> -- v3: win32u: Fix that the replaced font replaces other fonts. win32u: Fix the second_name parsed wrong. https://gitlab.winehq.org/wine/wine/-/merge_requests/628

3 4

Re: [PATCH v2 0/2] MR606: msvcp90: Call 'setlocale' before 'mbstowcs_s'
by Piotr Caban (＠piotr) Aug. 18, 2022

Aug. 18, 2022

On Wed Aug 17 06:20:22 2022 +0000, Yeshun Ye wrote: > I built the demo by vs2003/vs2005/vs2008, and it printed 'read: ok' for > vs2003 building, but printed 'open file failed!' for vs2005/vs2008 > buildings. > I think "\_MSVCP_VER >= 71" can change to "\_MSVCP_VER >= 80" at least. > And I think it's a bug on vs2005/vs2008, do we have to be exactly the > same as windows? Why do we keep this bug? I've created MR !673 to fix msvcp71. I've made a mistake while testing it earlier. The idea is to implement the function as closely to native as possible. Applications may depend on bugs in libraries (and taking into account _Fiopen is an internal function it's not necessarily a bug). -- https://gitlab.winehq.org/wine/wine/-/merge_requests/606#note_6472

1 0

[PATCH 0/1] MR673: msvcp71: Use MultiByteToWideChar in _Fiopen implemen.
by Piotr Caban (＠piotr) Aug. 18, 2022

Aug. 18, 2022

Also use MultiByteToWideChar in msvcp71. I've made a mistake while testing it earlier. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/673

2 1

[PATCH 0/5] MR665: win32u: Tweak reading / writing of DEVMODEW into the registry.
by Rémi Bernon Aug. 18, 2022

Aug. 18, 2022

Split from https://gitlab.winehq.org/wine/wine/-/merge_requests/576, for which I'll then add modes sorting, and move support for stretched / interlaced modes from winemac to win32u. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/665

2 10

[PATCH v3 0/1] MR672: Draft: Common item dialog fixes
by Ben Cottrell (＠bgc) Aug. 18, 2022

Aug. 18, 2022

Made changes to comdlg32/itemdlg.c to add missing controls, which are the address bar and the up button. Updated bitmaps in comctl32 and SVG files for history icons. My changes help to resolve this bug: https://bugs.winehq.org/show_bug.cgi?id=50338. -- v3: comdlg32: Add missing controls and their functionality to IFileDialog, or the common item dialog. https://gitlab.winehq.org/wine/wine/-/merge_requests/672

3 2

[PATCH 0/10] MR658: winepulse: Switch to mmdevapi's unixlib.h
by Davide Beatrici (＠davidebeatrici) Aug. 18, 2022

Aug. 18, 2022

Follow-up to !600. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/658

3 20