April 2025 - wine-gitlab

[PATCH 0/1] MR7940: dmsynth: Don't report an underrun when current equals write position
by Michael Stefaniuc (＠mstefani) 01 May '25

01 May '25

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

2 3

[PATCH v3 0/3] MR7934: mshtml: Validate builtin host functions in IE9+ using prototype_id rather than tid where possible.
by Gabriel Ivăncescu (＠insn) 01 May '25

01 May '25

-- v3: mshtml: Use designated initializers for the Location dispex data. mshtml: Validate builtin host functions in IE9+ using prototype_id rather mshtml: Store the object_id of the last object in the prototype chain that https://gitlab.winehq.org/wine/wine/-/merge_requests/7934

3 5

[PATCH 0/1] MR7586: cmd: Handle cntl-c events correctly
by Jeff Latimer (＠greyghoster) 01 May '25

01 May '25

https://bugs.winehq.org/show_bug.cgi?id=27605 describes how cntl-c is not handled well in cmd.exe, this patch allows the cntl-c to be returned in the buffer so that it can processed. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7586

3 9

[PATCH 0/2] MR7942: windowscodecs/tests: Add a test for 24bpp TIFF with separate sample planes.
by Nikolay Sivov (＠nsivov) 30 Apr '25

30 Apr '25

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

2 2

[PATCH 2/2] windowscodecs/tiff: Add support for files with separate planes.
by Nikolay Sivov 30 Apr '25

30 Apr '25

From: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/windowscodecs/libtiff.c | 115 +++++++++++++++++++++++--- dlls/windowscodecs/tests/tiffformat.c | 2 - 2 files changed, 104 insertions(+), 13 deletions(-) diff --git a/dlls/windowscodecs/libtiff.c b/dlls/windowscodecs/libtiff.c index 584dad116ed..5b5b6aa044a 100644 --- a/dlls/windowscodecs/libtiff.c +++ b/dlls/windowscodecs/libtiff.c @@ -152,6 +152,7 @@ typedef struct { UINT tile_size; int tiled; UINT tiles_across; + UINT tiles_along; } tiff_decode_info; struct tiff_decoder @@ -204,16 +205,11 @@ static HRESULT tiff_get_decode_info(TIFF *tiff, tiff_decode_info *decode_info) decode_info->samples = samples; if (samples == 1) - planar = 1; + planar = PLANARCONFIG_CONTIG; else { ret = TIFFGetField(tiff, TIFFTAG_PLANARCONFIG, &planar); - if (!ret) planar = 1; - if (planar != 1) - { - FIXME("unhandled planar configuration %u\n", planar); - return E_FAIL; - } + if (!ret) planar = PLANARCONFIG_CONTIG; } decode_info->planar = planar; @@ -475,6 +471,7 @@ static HRESULT tiff_get_decode_info(TIFF *tiff, tiff_decode_info *decode_info) decode_info->tile_stride = ((decode_info->frame.bpp * decode_info->tile_width + 7)/8); decode_info->tile_size = decode_info->tile_height * decode_info->tile_stride; decode_info->tiles_across = (decode_info->frame.width + decode_info->tile_width - 1) / decode_info->tile_width; + decode_info->tiles_along = (decode_info->frame.height + decode_info->tile_height - 1) / decode_info->tile_height; } else if ((ret = TIFFGetField(tiff, TIFFTAG_ROWSPERSTRIP, &decode_info->tile_height))) { @@ -656,19 +653,115 @@ static HRESULT CDECL tiff_decoder_get_decoder_palette(struct decoder *iface, UIN return WINCODEC_ERR_PALETTEUNAVAILABLE; } -static HRESULT tiff_decoder_read_tile(struct tiff_decoder *This, UINT tile_x, UINT tile_y) +static HRESULT tiff_read_tile(struct tiff_decoder *decoder, UINT tile_x, UINT tile_y, UINT s, void *buffer, + size_t size, size_t *out_length) { + tiff_decode_info *info = &decoder->cached_decode_info; + unsigned int index; tsize_t ret; - tiff_decode_info *info = &This->cached_decode_info; + + *out_length = 0; if (info->tiled) - ret = TIFFReadEncodedTile(This->tiff, tile_x + tile_y * info->tiles_across, This->cached_tile, info->tile_size); + { + index = tile_x + tile_y * info->tiles_across + info->tiles_across * info->tiles_along * s; + ret = TIFFReadEncodedTile(decoder->tiff, index, buffer, size); + } else - ret = TIFFReadEncodedStrip(This->tiff, tile_y, This->cached_tile, info->tile_size); + { + index = TIFFComputeStrip(decoder->tiff, tile_y * info->tile_height, s); + ret = TIFFReadEncodedStrip(decoder->tiff, index, buffer, size); + } if (ret == -1) return E_FAIL; + *out_length = ret; + return S_OK; +} + +static HRESULT tiff_decoder_copy_planes(struct tiff_decoder *decoder, const uint8_t *srcdata) +{ + tiff_decode_info *info = &decoder->cached_decode_info; + size_t sample_size, width_bytes; + const uint8_t *src; + uint8_t *dst; + + if (info->bps % 8) + { + FIXME("Separate sample planes for %ubps are not supported.\n", info->bps); + return E_NOTIMPL; + } + + sample_size = info->bps / 8; + width_bytes = info->tile_width * sample_size; + + for (int y = 0; y < info->tile_height; y++) + { + dst = decoder->cached_tile + y * width_bytes * info->samples; + + for (int x = 0; x < info->tile_width; x++) + { + for (int s = 0; s < info->samples; s++) + { + src = srcdata + (y + s * info->tile_height) * width_bytes + x * sample_size; + + if (info->bps == 8) + *dst = *src; + else if (info->bps == 16) + *(uint16_t *)dst = *(uint16_t *)src; + else if (info->bps == 32) + *(uint32_t *)dst = *(uint32_t *)src; + else if (info->bps == 64) + *(uint64_t *)dst = *(uint64_t *)src; + else + memcpy(dst, src, sample_size); + + dst += sample_size; + } + } + } + + return S_OK; +} + +static HRESULT tiff_decoder_read_tile(struct tiff_decoder *This, UINT tile_x, UINT tile_y) +{ + tiff_decode_info *info = &This->cached_decode_info; + size_t len; + HRESULT hr; + + if (info->planar == PLANARCONFIG_CONTIG) + { + hr = tiff_read_tile(This, tile_x, tile_y, 0, This->cached_tile, info->tile_size, &len); + } + else + { + size_t offset = 0, size = info->samples * info->tile_size; + uint8_t *buffer; + + /* Read separate planes one by one, and put them together in contiguous form. */ + if (!(buffer = malloc(size))) + return E_OUTOFMEMORY; + + for (int s = 0; s < info->samples; ++s) + { + hr = tiff_read_tile(This, tile_x, tile_y, s, buffer + offset, size, &len); + if (FAILED(hr)) break; + + offset += len; + size -= len; + } + + if (SUCCEEDED(hr)) + hr = tiff_decoder_copy_planes(This, buffer); + + free(buffer); + } + + if (FAILED(hr)) + return hr; + /* 3bps RGB */ if (info->source_bpp == 3 && info->samples == 3 && info->frame.bpp == 24) { diff --git a/dlls/windowscodecs/tests/tiffformat.c b/dlls/windowscodecs/tests/tiffformat.c index b292fd4cabd..49323028857 100644 --- a/dlls/windowscodecs/tests/tiffformat.c +++ b/dlls/windowscodecs/tests/tiffformat.c @@ -1626,9 +1626,7 @@ static void test_tiff_24bpp_separate_planes(void) HRESULT hr; hr = create_decoder(&tiff_24bpp_separate_planes_data, sizeof(tiff_24bpp_separate_planes_data), &decoder); - todo_wine ok(hr == S_OK, "Failed to load TIFF image data %#lx\n", hr); - if (FAILED(hr)) return; ok(decoder != NULL, "Failed to load TIFF image data\n"); hr = IWICBitmapDecoder_GetFrameCount(decoder, &count); -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/7942

0 -1

Re: [PATCH v2 0/5] MR7933: win32u: Add procedure loading to generic OpenGL code. - approved
by Alexandre Julliard (＠julliard) 30 Apr '25

30 Apr '25

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

1 0

Re: [PATCH v2 0/1] MR7905: krnl386: Set carry flag for unimplemented DPMI 0800h call (Physical Address Mapping) - approved
by Alexandre Julliard (＠julliard) 30 Apr '25

30 Apr '25

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

1 0

Re: [PATCH v2 0/4] MR7822: wined3d: Move FFP extra states away from STATE_RENDER, part 3/4. - approved
by Alexandre Julliard (＠julliard) 30 Apr '25

30 Apr '25

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

1 0

Re: [PATCH v2 0/4] MR7939: ntdll: Implement ProcessBasePriority class in NtSetInformationProcess. - approved
by Alexandre Julliard (＠julliard) 30 Apr '25

30 Apr '25

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

1 0

Re: [PATCH v4 0/1] MR7925: gdi32: Limit source string length in logfont_AtoW (ASan). - approved
by Alexandre Julliard (＠julliard) 30 Apr '25

30 Apr '25

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

1 0

2025

2024

2023

2022

wine-gitlab April 2025