On Mon Jul 10 16:57:51 2023 +0000, Alex Henrie wrote:
> If thread safety matters here, could you use strtok_r?
I'm not sure if it does, just wanted to make sure there is no race conditions. Not sure why I missed strtok_r, thanks.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3259#note_38565
I'm aware that winecfg allows unlinking of symlinked folders, but this is very annoying when regularly recreating WINEPREFIXES. This change makes it easy to isolate those folders from the start.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3259
> we should test the behavior of ctrl-c
I'm bit late to be reporting this :smile:, but Ctrl-C in (probably all) `-mwindows` Wine apps is now ignored (`winecfg`, `regedit`, etc). On the Windows apps side of things, at least Notepad++ also doesn't close on Ctrl-C.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3145#note_38548
--
v4: vkd3d-shader/dxil: Read function bodies.
vkd3d-shader/dxil: Read numeric constants.
vkd3d-shader/dxil: Read global function declarations.
vkd3d-shader/dxil: Validate the module format version.
vkd3d-shader/dxil: Read the value symbol table.
vkd3d-shader/dxil: Read the type table.
vkd3d-shader/dxil: Use size_t where applicable.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/259
On Mon Jul 10 15:59:56 2023 +0000, Jakub Staroń wrote:
> It is indeed Wine-specific, but on Windows it isn't as much of a problem
> because AllocConsole gives you the same cmd you would use normally,
> while wineconsole is inferior to a normal terminal. Also is there any
> reason to explicitly block console output on GUI apps? If I understand
> it correctly on Windows it's done because otherwise a new console window
> is created, wine doesn't do that though?
all these four configurations should behave the same way regarding log output:
1) app on wine being run from a unix shell
2) app on wine being from with wineconsole
3) app on wine being run from a cmd window
4) app on windows being run from a cmd window
if you see a discrepancy in output between 2 of these four items, please report (or even better file a bug report)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3145#note_38542
On Mon Jul 10 14:53:19 2023 +0000, Jacek Caban wrote:
> Is it some Wine-specific use case? I think that something like that
> would not work on Windows.
It is indeed Wine-specific, but on Windows it isn't as much of a problem because AllocConsole gives you the same cmd you would use normally, while wineconsole is inferior to a normal terminal. Also is there any reason to explicitly block console output on GUI apps? If I understand it correctly on Windows it's done because otherwise a new console window is created, wine doesn't do that though?
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3145#note_38539
On Mon Jul 10 14:53:19 2023 +0000, Jakub Staroń wrote:
> I know it's relying on broken behavior but before this PR it was nice to
> be able to log to the terminal from GUI apps (!3022 would be a much
> nicer solution for that). Does anyone know if there is a workaround to
> be still able to do that? My main issue is that wineconsole is way
> slower than normal terminal emulators.
Is it some Wine-specific use case? I think that something like that would not work on Windows.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3145#note_38531
I know it's relying on broken behavior but before this PR it was nice to be able to log to the terminal from GUI apps (!3022 would be a much nicer solution for that). Does anyone know if there is a workaround to be still able to do that? My main issue is that wineconsole is way slower than normal terminal emulators.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3145#note_38528
Fix the issue of incorrect linkid calculation due to a lack of judgment on the current type.
Signed-off-by: Zhaoyi <zhaoyi(a)uniontech.com>
--
v10: comctl32/tests: Add a test case to get syslinkid.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3114
On Sat Jul 8 07:23:37 2023 +0000, Rémi Bernon wrote:
> You should only call `WindowsDeleteString` if the refcount gets to 0
> (and then you should also free the hostname, which I also missed).
Yeah, sorry for missing that. I've just been very burned out with life in general.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3249#note_38470
On Mon Jul 10 01:45:21 2023 +0000, Zhao Yi wrote:
> Thank you for reviewing my code and I will fix the above issue later.
There is no need to rush this. Please take your time to carefully review the code by yourself before submitting it for review. Look out for coding style issues and review each line you've changed. Overall, the fix looks good. Thanks for your work.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3114#note_38455
Zhiyi Zhang (@zhiyi) commented about dlls/comctl32/tests/syslink.c:
> }
>
> defwndproc_counter++;
> - ret = DefWindowProcW(hwnd, message, wParam, lParam);
> + switch(message)
You only need to record nml->item.iLink so you don't really need to process the message. So you can move this switch statement before defwndproc_counter++ and let the DefWindowProcW() handle it. Basically, add a switch statement, record nml->item.iLink from WM_NOTIFY, and break on other messages.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3114#note_38452
Fixes regression introduced by 35c8f15bc67f7602438fe0c39a69ff6ddf6b8297
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55215
We already test for an invalid pointer, those tests don't really help.
The test against DrawTextA crashes on windows sometimes, the other test shows that windows probably just overruns the stack variable:
`textheight = DrawTextExA((HDC)0xdeadbeef, (LPSTR)0xdeadbeef, 100, &rect, 0, 0);`
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3276
This MR attempts to bring Wine's handling of `setlocale` and `_create_locale` more in line with the behavior of native >= `msvcr110`. It does this by replacing the usage of the LCID based `GetLocaleInfo` with the sname based `GetLocaleInfoEx`. `GetLocaleInfo` doesn't support neutral locales, even on native, which causes problems with neutral Chinese locales like `zh-Hans` and `zh-Hant`.
It also improves the accuracy of the internal `__lc_locale_name_func`, as tests indicate that it returns `LOCALE_SNAME`s instead of ISO 639 language names.
Potential future improvements:
* Modifying the generation of the `pclmap` and `pcumap` fields of `threadlocaleinfostruct` to work on WCHARs.
* I've done some changes to `__thread_data` which make it not fully match native, I'll submit a follow up which fixes this.
--
v4: make_unicode: rename Hong Kong SAR China to Hong Kong SAR to match native behavior.
msvcrt: Use GetLocaleInfoEx to compare locale info.
msvcrt: Remap synonyms to snames.
msvcrt: Skip exhaustive locale search with valid snames.
msvcrt: Convert locale_to_LCID to snames.
msvcrt: Use snames instead of LCIDs in create_locinfo.
msvcr120/tests: Check ___lc_locale_name_func with neutral Chinese locales.
msvcr110/tests: Add tests for new setlocale behaviors.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3050
--
v2: Release 8.0.2.
winedump: Fix out of bounds access (spotted by GCC13).
wordpad: Resize buttons of toolbar to make them align with height of the font list combox.
winemenubuilder: Skip desktop integration for certain associations.
wine.inf: Put the Clients key in the right place.
winemac: Initialize retina_on to avoid incorrect cached display data in high-res/retina mode.
winemac.drv: Force a window in front of its peers if its level is decreased.
threadpoolwinrt: Forward a Release() to a Release() method.
ntdll: Avoid integer overflow in block_get_subheap().
mfreadwrite: Fix an address of operator typo.
winemac: Don't constrain surface dimensions to the onscreen part of a window.
winemac: Set the Cocoa window contentMaxSize to the size limits from constrain_window_frame().
winemac: Refactor constrain_window_frame() to use separate origin and size arguments.
include: Fix IDirectXVideoDecoderService declaration in dxva2api.idl.
include: Fix c++ compiler error with msxml2.idl.
activeds: Free memory on error paths (coverity).
mshtml: Store minimum compat mode required for events in the ctor table.
mshtml: Fix URI leak in NewURI on failure.
mshtml: Fix nsIFile dir leak in init_xpcom.
mshtml: Fix factory leak in init_nsio.
winemac: Fix memory leak in UpdateDisplayDevices.
mshtml: Fix nsparser leak if script_elem_from_nsscript failed.
mshtml: Fix nsuri leak if create_channelbsc failed.
mshtml: Fix nsscript leak in bind_event_scripts.
mshtml: Fix nsevent leak in create_document_event.
evr: Fix incorrect integral computation.
msvcrt: Fix crash when rethrowing after a non-C++ exception on x86_64.
winedbg: Don't crash when no search path has been set.
ntdll: Open application manifest files with FILE_SHARE_DELETE.
kernel32/tests: Test for manifest file open share mode in CreateActCtxW.
wined3d: Add GPU information for AMD Raphael.
shlwapi: Implement StrFormatByteSizeEx.
ntdll: Fix typo in the check for hidden files.
ntdll: Set xattr in NtCreateFile if inferred and requested attributes don't match.
ntdll: Only infer hidden attribute from file name if xattr is not present.
ntdll: Handle hidden file names inside get_file_info instead of after it.
ntdll/tests: Add test for file attributes of files with names beginning with a dot.
ntdll: Make fd_set_file_info() static.
mfplat: Implement MFCreatePathFromURL().
dxdiag: Ignore option /64bit on the commandline.
findstr: Add basic functionality.
include/msvcrt: Add __WINE_(DEALLOC|MALLOC) attributes to _strdup and _wcsdup.
This merge request has too many patches to be relayed via email.
Please visit the URL below to see the contents of the merge request.
https://gitlab.winehq.org/wine/wine/-/merge_requests/3168
In HttpSendRequestW and HttpSendRequestExW, if the header pointer is not
null but the length parameter is 0, the header length should be derived
from the string length instead.
In HttpSendRequestA and HttpSendRequestExA, on the same scenario, the
function should fail instead.
--
v2: wininet: Handle http headers correctly when length is 0
https://gitlab.winehq.org/wine/wine/-/merge_requests/3269