Search names in fonts in the order of Microsoft, Mac and finally Unicode platform. This is also the
order win32u uses to load font names.
Fix Granado Espada Japan (1219160) launcher crashes at start in the Japanese locale. The game ships
a font with a broken name record of Mac platform and encoding ID 0 (Roman) but the name string is in
code page 10001 (Japanese). This broken name record is placed before the name records for the
Microsoft platform so it gets selected first. Then the name …
[View More]string in the name record doesn't get
converted correctly to Unicode because of the wrong code page. Thus the EnumFontFamiliesExW()
in GdipPrivateAddMemoryFont() fails to find the font and causes game crash.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5421
[View Less]
On Thu Apr 4 08:24:18 2024 +0000, Zhiyi Zhang wrote:
> Also, in the commit message subject. Not just the MR subject.
OK, I think I got it ! I'll keep these in mind for next time.
[image](/uploads/233f99f4dd99c94026b9161be7194eb4/image.png)
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5427#note_66885
On Thu Apr 4 08:24:18 2024 +0000, Sam Joan Roque-Worcel wrote:
> Woops! That's what you get for coding late at night. I've hopefully
> addressed it now
We usually add a dot at the end of the message subject. Please do that as well.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5427#note_66876
Addresses this issue: https://bugs.winehq.org/show_bug.cgi?id=53985#add_comment
This is done by adding three characters into Tahoma that are missing. I achieved this by replacing the characters with similar shape from within the same font (e.g. + for U+FF0B) or by getting the font from a font that had an open source license (liberation2) in the case of U+25BC.
Once I had that done, I regenerated the font by running the following command:
```
fontforge -script fonts/genttf.ff fonts/tahoma.sfd …
[View More]fonts/tahoma.ttf
```
The image below shows the issue as reported by gatlibs:

The image below shows the fix:

--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5447
[View Less]
--
v2: win32u: Split virtual desktop modes to a add_virtual_modes helper.
win32u: Delay writing current display settings to the registry.
winex11: Let win32u decide when to force update the display cache.
win32u: Don't force refresh the display cache on thread desktop change.
winex11: Report all sources as detached in virtual desktop mode.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5422
The call in freetype.c:expand_mac_font() would be a good candidate for asprintf(), but I'm planning to send a patch soon to remove that resource-fork font extraction anyway.
--
v2: win32u: Replace sprintf with snprintf to avoid deprecation warnings on macOS.
win32u: Use PATH_MAX for Unix paths instead of MAX_PATH (from Win32).
win32u: Enlarge buffer size in format_date.
win32u: Enlarge buffer size in _CDS_flags.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5436
On Thu Apr 4 08:24:18 2024 +0000, Sam Joan Roque-Worcel wrote:
> Ah! Thank you! I've added that in there now. Thank you for the reminder.
I meant the commit message subject, not the merge request subject. You can use 'git commit --amend' and then force push.
And please squash those two patches into one.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5427#note_66777
On Wed Apr 3 09:52:58 2024 +0000, Sam Joan Roque-Worcel wrote:
> changed this line in [version 2 of the diff](/wine/wine/-/merge_requests/5427/diffs?diff_id=108367&start_sha=a41a4147bacc2bd3a64d582fb8918630ff795f1d#1309fd48153be56cdd4228d6e4bc8022a837b2de_39_39)
Good spotting! No worries, thank you for taking the time to review it!
I've changed the value to 17 :thumbsup:
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5427#note_66770
--
v3: widl: Add support for VT_USERDEFINED to SLTG typelib generator.
widl: Properly align name table entries.
widl: Add support for structures.
widl: Add more traces.
widl: Use generic 'struct sltg_data' for collecting data.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5397
The call in freetype.c:expand_mac_font() would be a good candidate for asprintf(), but I'm planning to send a patch soon to remove that resource-fork font extraction anyway.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5436
On Tue Apr 2 07:23:54 2024 +0000, Aida Jonikienė wrote:
> I wonder if the lack of tests could be a problem merging this (I've been
> thinking of using a TRY/EXCEPT block to test the behavior of this flag
> but this might not be right)
I don't think tests are useful here.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5437#note_66713
Follow up to !5342. This adds the last missing piece for fixing how `ShellExecute` finds files, and restore the ability of running native unix programs with `ShellExecute`
--
v2: shell32: Accept ".so" as a valid program extension.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5400
First change also avoids a leak of classes (later activates) when the transform is successfully connected.
The use of MFTEnumEx is necessary so that the tests work, as they use a locally registered transform.
--
v2: mfreadwrite/tests: Test the D3D awareness of source reader transforms.
mfreadwrite/tests: Test the source reader stream change events.
mfreadwrite/reader: Make the GetTransformForStream category parameter optional.
mfreadwrite/reader: Use MFTEnumEx to enumerate …
[View More]stream transforms.
https://gitlab.winehq.org/wine/wine/-/merge_requests/5412
[View Less]
Thanks for the update. It doesn't happen automatically, no. It's manually merged from time to time, and I think only at a point of tagged releases. For FAudio that would mean next 24.05 release, in about a month.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/5424#note_66634
This finally solves a FIXME that was added back in 1997.
NFS High Stakes (and Motor City Online) both emit a 0x190 access
violation on startup (according to game's strings this was meant
to be debugger detection; I think this was done to interfere in
a DRM bypass but I'm probably wrong).
But anyway that feature causes a Wine Debugger dialog to appear
when starting the games (if you click "Show Details" though they
suddenly start working normally but this is an extra annoyance
for average …
[View More]people who just want to play the game).
This change prevents that by not starting the debugger if the
SEM_NOGPFAULTERRORBOX flag is set (if that flag wasn't set then
people would be complaining about the game crashing every time)
which lets the exception silently go through (it can still be seen
with WINEDEBUG=+seh set though).
I'm not sure how I can test for this exception handling behavior
(I'm thinking of calling RaiseException() inside a __try/__except
block but that might not be exactly what the games are doing).
This patch was originally posted by Zeb in a diff form (so the
patch description has been written by me, DodoGTA).
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4567
[View Less]
This patch set now has 24 test failures in wmvcore/tests/wmvcore.c: test_async_reader_streaming(). So it won't be merged now. But I'd like to have it reviewed to see if I was doing things in the right direction.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/3193