February 2022 - Wine-Commits

Zebediah Figura : wined3d: Allocate texture BOs from heap.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: aa9b1fb3a3bb344b0f4f697203bac34a420aed4a URL: https://source.winehq.org/git/wine.git/?a=commit;h=aa9b1fb3a3bb344b0f4f6972… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Feb 23 20:21:41 2022 -0600 wined3d: Allocate texture BOs from heap. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/texture.c | 45 +++++++++++++++++++++++++++++++----------- dlls/wined3d/wined3d_private.h | 7 +------ 2 files changed, 34 insertions(+), 18 deletions(-) diff --git a/dlls/wined3d/texture.c b/dlls/wined3d/texture.c index 13c26692f77..75997914d77 100644 --- a/dlls/wined3d/texture.c +++ b/dlls/wined3d/texture.c @@ -745,7 +745,7 @@ void wined3d_texture_get_bo_address(const struct wined3d_texture *texture, if (location == WINED3D_LOCATION_BUFFER) { data->addr = NULL; - data->buffer_object = &sub_resource->bo.b; + data->buffer_object = sub_resource->bo; } else { @@ -903,12 +903,15 @@ static void wined3d_texture_get_memory(struct wined3d_texture *texture, unsigned static void wined3d_texture_remove_buffer_object(struct wined3d_texture *texture, unsigned int sub_resource_idx, struct wined3d_context_gl *context_gl) { - struct wined3d_bo_gl *bo = &texture->sub_resources[sub_resource_idx].bo.gl; + struct wined3d_texture_sub_resource *sub_resource = &texture->sub_resources[sub_resource_idx]; + struct wined3d_bo_gl *bo_gl = wined3d_bo_gl(sub_resource->bo); TRACE("texture %p, sub_resource_idx %u, context_gl %p.\n", texture, sub_resource_idx, context_gl); - wined3d_context_gl_destroy_bo(context_gl, bo); + wined3d_context_gl_destroy_bo(context_gl, bo_gl); wined3d_texture_invalidate_location(texture, sub_resource_idx, WINED3D_LOCATION_BUFFER); + sub_resource->bo = NULL; + heap_free(bo_gl); } static void wined3d_texture_unload_location(struct wined3d_texture *texture, @@ -2059,16 +2062,23 @@ static void wined3d_texture_gl_prepare_buffer_object(struct wined3d_texture_gl * struct wined3d_bo_gl *bo; sub_resource = &texture_gl->t.sub_resources[sub_resource_idx]; - bo = &sub_resource->bo.gl; - if (bo->id) + + if (sub_resource->bo) + return; + + if (!(bo = heap_alloc(sizeof(*bo)))) return; if (!wined3d_device_gl_create_bo(wined3d_device_gl(texture_gl->t.resource.device), context_gl, sub_resource->size, GL_PIXEL_UNPACK_BUFFER, GL_STREAM_DRAW, true, GL_MAP_READ_BIT | GL_MAP_WRITE_BIT | GL_CLIENT_STORAGE_BIT, bo)) + { + heap_free(bo); return; + } TRACE("Created buffer object %u for texture %p, sub-resource %u.\n", bo->id, texture_gl, sub_resource_idx); + sub_resource->bo = &bo->b; } static void wined3d_texture_force_reload(struct wined3d_texture *texture) @@ -3249,7 +3259,7 @@ static BOOL wined3d_texture_gl_load_texture(struct wined3d_texture_gl *texture_g /* Don't use PBOs for converted surfaces. During PBO conversion we look at * WINED3D_TEXTURE_CONVERTED but it isn't set (yet) in all cases it is * getting called. */ - if (conversion && sub_resource->bo.gl.id) + if (conversion && sub_resource->bo) { TRACE("Removing the pbo attached to texture %p, %u.\n", texture_gl, sub_resource_idx); @@ -3386,7 +3396,7 @@ static void wined3d_texture_gl_unload_location(struct wined3d_texture *texture, sub_count = texture->level_count * texture->layer_count; for (i = 0; i < sub_count; ++i) { - if (texture_gl->t.sub_resources[i].bo.gl.id) + if (texture_gl->t.sub_resources[i].bo) wined3d_texture_remove_buffer_object(&texture_gl->t, i, context_gl); } break; @@ -5294,18 +5304,24 @@ static BOOL wined3d_texture_vk_prepare_buffer_object(struct wined3d_texture_vk * struct wined3d_bo_vk *bo; sub_resource = &texture_vk->t.sub_resources[sub_resource_idx]; - bo = &sub_resource->bo.vk; - if (bo->vk_buffer) + if (sub_resource->bo) return TRUE; + if (!(bo = heap_alloc(sizeof(*bo)))) + return FALSE; + if (!wined3d_context_vk_create_bo(context_vk, sub_resource->size, VK_BUFFER_USAGE_TRANSFER_DST_BIT | VK_BUFFER_USAGE_TRANSFER_SRC_BIT, VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT, bo)) + { + heap_free(bo); return FALSE; + } /* Texture buffer objects receive a barrier to HOST_READ in wined3d_texture_vk_download_data(), * so they don't need it when they are mapped for reading. */ bo->host_synced = true; + sub_resource->bo = &bo->b; TRACE("Created buffer object %p for texture %p, sub-resource %u.\n", bo, texture_vk, sub_resource_idx); return TRUE; } @@ -5380,10 +5396,15 @@ static void wined3d_texture_vk_unload_location(struct wined3d_texture *texture, sub_count = texture->level_count * texture->layer_count; for (i = 0; i < sub_count; ++i) { - if (texture->sub_resources[i].bo.vk.vk_buffer) + struct wined3d_texture_sub_resource *sub_resource = &texture->sub_resources[i]; + + if (sub_resource->bo) { - wined3d_context_vk_destroy_bo(context_vk, &texture->sub_resources[i].bo.vk); - texture->sub_resources[i].bo.vk.vk_buffer = VK_NULL_HANDLE; + struct wined3d_bo_vk *bo_vk = wined3d_bo_vk(sub_resource->bo); + + wined3d_context_vk_destroy_bo(context_vk, bo_vk); + heap_free(bo_vk); + sub_resource->bo = NULL; } } break; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 6fd51b77e2a..a9837911ea6 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -4532,12 +4532,7 @@ struct wined3d_texture unsigned int map_count; uint32_t map_flags; DWORD locations; - union - { - struct wined3d_bo b; - struct wined3d_bo_gl gl; - struct wined3d_bo_vk vk; - } bo; + struct wined3d_bo *bo; void *user_memory; } *sub_resources;

1 0

Zebediah Figura : winegstreamer: Flatten wg_parser_event into a wg_parser_buffer structure.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: a4be9e85687f7a8d7476ee3d3ced80eeec720f57 URL: https://source.winehq.org/git/wine.git/?a=commit;h=a4be9e85687f7a8d7476ee3d… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Feb 23 15:46:50 2022 -0600 winegstreamer: Flatten wg_parser_event into a wg_parser_buffer structure. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/gst_private.h | 2 +- dlls/winegstreamer/main.c | 8 +++--- dlls/winegstreamer/media_source.c | 18 ++++++------- dlls/winegstreamer/quartz_parser.c | 38 +++++++++++++-------------- dlls/winegstreamer/unixlib.h | 31 +++++++--------------- dlls/winegstreamer/wg_parser.c | 54 +++++++++++++++++--------------------- dlls/winegstreamer/wm_reader.c | 38 +++++++++++++-------------- 7 files changed, 84 insertions(+), 105 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=a4be9e85687f7a8d7476…

1 0

Zebediah Figura : winegstreamer: Remove support for flushing the wg_parser object.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: 5144b27661fcd6705353d832e0383085f8afe842 URL: https://source.winehq.org/git/wine.git/?a=commit;h=5144b27661fcd6705353d832… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Feb 23 15:46:47 2022 -0600 winegstreamer: Remove support for flushing the wg_parser object. Aside from EOS logic, which is now handled entirely on the client side, wg_parser_stream_get_event() now only waits for data processing—that is, demuxing, decoding, and format conversion. While unblocking waits in wg_parser_stream_get_event() does allow that function to return immediately, a subsequent seek request in GStreamer will still have to wait for that data processing to complete and for the stream thread to return to the demuxer's main loop. In essence, wg_parser_begin_flush() is only moving costs around. In theory we could force the GStreamer pipeline to complete faster by actually flushing it. In practice this isn't really true. Individual elements do check whether they are flushing before processing, but even elements which take a relatively long time (i.e. multiple milliseconds) to process data don't periodically check whether they are flushing while doing so. Although there is arguably a benefit to skipping some elements by flushing the GStreamer pipeline, it does not seem worth the added code complexity in Wine. The real point of flushing in DirectShow or GStreamer is to unblock long or unbounded waits in sink elements (i.e. waits for PTS, or waits for running state while rendering preroll frames). None of these waits apply here. Waits for actual sample processing complete in bounded time, and should ideally take less than the sample DTS to complete (or we are already in trouble). Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/gst_private.h | 5 +---- dlls/winegstreamer/main.c | 14 ++---------- dlls/winegstreamer/media_source.c | 6 +----- dlls/winegstreamer/quartz_parser.c | 12 +---------- dlls/winegstreamer/unixlib.h | 3 --- dlls/winegstreamer/wg_parser.c | 44 ++------------------------------------ dlls/winegstreamer/wm_reader.c | 7 +----- 7 files changed, 8 insertions(+), 83 deletions(-) diff --git a/dlls/winegstreamer/gst_private.h b/dlls/winegstreamer/gst_private.h index a63daaf04b9..288d127b23b 100644 --- a/dlls/winegstreamer/gst_private.h +++ b/dlls/winegstreamer/gst_private.h @@ -70,9 +70,6 @@ void wg_parser_destroy(struct wg_parser *parser); HRESULT wg_parser_connect(struct wg_parser *parser, uint64_t file_size); void wg_parser_disconnect(struct wg_parser *parser); -void wg_parser_begin_flush(struct wg_parser *parser); -void wg_parser_end_flush(struct wg_parser *parser); - bool wg_parser_get_next_read_offset(struct wg_parser *parser, uint64_t *offset, uint32_t *size); void wg_parser_push_data(struct wg_parser *parser, const void *data, uint32_t size); @@ -83,7 +80,7 @@ void wg_parser_stream_get_preferred_format(struct wg_parser_stream *stream, stru void wg_parser_stream_enable(struct wg_parser_stream *stream, const struct wg_format *format); void wg_parser_stream_disable(struct wg_parser_stream *stream); -bool wg_parser_stream_get_event(struct wg_parser_stream *stream, struct wg_parser_event *event); +void wg_parser_stream_get_event(struct wg_parser_stream *stream, struct wg_parser_event *event); bool wg_parser_stream_copy_buffer(struct wg_parser_stream *stream, void *data, uint32_t offset, uint32_t size); void wg_parser_stream_release_buffer(struct wg_parser_stream *stream); diff --git a/dlls/winegstreamer/main.c b/dlls/winegstreamer/main.c index f85e9995525..13db07c6edb 100644 --- a/dlls/winegstreamer/main.c +++ b/dlls/winegstreamer/main.c @@ -96,16 +96,6 @@ void wg_parser_disconnect(struct wg_parser *parser) __wine_unix_call(unix_handle, unix_wg_parser_disconnect, parser); } -void wg_parser_begin_flush(struct wg_parser *parser) -{ - __wine_unix_call(unix_handle, unix_wg_parser_begin_flush, parser); -} - -void wg_parser_end_flush(struct wg_parser *parser) -{ - __wine_unix_call(unix_handle, unix_wg_parser_end_flush, parser); -} - bool wg_parser_get_next_read_offset(struct wg_parser *parser, uint64_t *offset, uint32_t *size) { struct wg_parser_get_next_read_offset_params params = @@ -182,7 +172,7 @@ void wg_parser_stream_disable(struct wg_parser_stream *stream) __wine_unix_call(unix_handle, unix_wg_parser_stream_disable, stream); } -bool wg_parser_stream_get_event(struct wg_parser_stream *stream, struct wg_parser_event *event) +void wg_parser_stream_get_event(struct wg_parser_stream *stream, struct wg_parser_event *event) { struct wg_parser_stream_get_event_params params = { @@ -190,7 +180,7 @@ bool wg_parser_stream_get_event(struct wg_parser_stream *stream, struct wg_parse .event = event, }; - return !__wine_unix_call(unix_handle, unix_wg_parser_stream_get_event, &params); + __wine_unix_call(unix_handle, unix_wg_parser_stream_get_event, &params); } bool wg_parser_stream_copy_buffer(struct wg_parser_stream *stream, diff --git a/dlls/winegstreamer/media_source.c b/dlls/winegstreamer/media_source.c index 32006964f3b..694497da259 100644 --- a/dlls/winegstreamer/media_source.c +++ b/dlls/winegstreamer/media_source.c @@ -388,7 +388,6 @@ static void start_pipeline(struct media_source *source, struct source_async_comm if (position->vt == VT_I8) wg_parser_stream_seek(source->streams[0]->wg_stream, 1.0, position->hVal.QuadPart, 0, AM_SEEKING_AbsolutePositioning, AM_SEEKING_NoPositioning); - wg_parser_end_flush(source->wg_parser); for (i = 0; i < source->stream_count; i++) flush_token_queue(source->streams[i], position->vt == VT_EMPTY); @@ -416,8 +415,6 @@ static void stop_pipeline(struct media_source *source) { unsigned int i; - wg_parser_begin_flush(source->wg_parser); - for (i = 0; i < source->stream_count; i++) { struct media_stream *stream = source->streams[i]; @@ -543,8 +540,7 @@ static void wait_on_sample(struct media_stream *stream, IUnknown *token) for (;;) { - if (!wg_parser_stream_get_event(stream->wg_stream, &event)) - return; + wg_parser_stream_get_event(stream->wg_stream, &event); TRACE("Got event of type %#x.\n", event.type); diff --git a/dlls/winegstreamer/quartz_parser.c b/dlls/winegstreamer/quartz_parser.c index 30afd0874ea..628758b838b 100644 --- a/dlls/winegstreamer/quartz_parser.c +++ b/dlls/winegstreamer/quartz_parser.c @@ -833,11 +833,7 @@ static DWORD CALLBACK stream_thread(void *arg) continue; } - if (!wg_parser_stream_get_event(pin->wg_stream, &event)) - { - LeaveCriticalSection(&pin->flushing_cs); - continue; - } + wg_parser_stream_get_event(pin->wg_stream, &event); TRACE("Got event of type %#x.\n", event.type); @@ -971,7 +967,6 @@ static HRESULT parser_init_stream(struct strmbase_filter *iface) return S_OK; filter->streaming = true; - wg_parser_end_flush(filter->wg_parser); /* DirectShow retains the old seek positions, but resets to them every time * it transitions from stopped -> paused. */ @@ -1011,7 +1006,6 @@ static HRESULT parser_cleanup_stream(struct strmbase_filter *iface) return S_OK; filter->streaming = false; - wg_parser_begin_flush(filter->wg_parser); for (i = 0; i < filter->source_count; ++i) { @@ -1336,8 +1330,6 @@ static HRESULT WINAPI GST_Seeking_SetPositions(IMediaSeeking *iface, if (!(current_flags & AM_SEEKING_NoFlush)) { - wg_parser_begin_flush(filter->wg_parser); - for (i = 0; i < filter->source_count; ++i) { if (filter->sources[i]->pin.pin.peer) @@ -1365,8 +1357,6 @@ static HRESULT WINAPI GST_Seeking_SetPositions(IMediaSeeking *iface, if (!(current_flags & AM_SEEKING_NoFlush)) { - wg_parser_end_flush(filter->wg_parser); - for (i = 0; i < filter->source_count; ++i) { struct parser_source *flush_pin = filter->sources[i]; diff --git a/dlls/winegstreamer/unixlib.h b/dlls/winegstreamer/unixlib.h index f445fc7ac7e..1ef80991fb4 100644 --- a/dlls/winegstreamer/unixlib.h +++ b/dlls/winegstreamer/unixlib.h @@ -238,9 +238,6 @@ enum unix_funcs unix_wg_parser_connect, unix_wg_parser_disconnect, - unix_wg_parser_begin_flush, - unix_wg_parser_end_flush, - unix_wg_parser_get_next_read_offset, unix_wg_parser_push_data, diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index 85c28895159..61120a72da8 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -87,7 +87,7 @@ struct wg_parser GstFlowReturn ret; } read_request; - bool flushing, sink_connected; + bool sink_connected; bool unlimited_buffering; }; @@ -127,35 +127,6 @@ static NTSTATUS wg_parser_get_stream(void *args) return S_OK; } -static NTSTATUS wg_parser_begin_flush(void *args) -{ - struct wg_parser *parser = args; - unsigned int i; - - pthread_mutex_lock(&parser->mutex); - parser->flushing = true; - pthread_mutex_unlock(&parser->mutex); - - for (i = 0; i < parser->stream_count; ++i) - { - if (parser->streams[i]->enabled) - pthread_cond_signal(&parser->streams[i]->event_cond); - } - - return S_OK; -} - -static NTSTATUS wg_parser_end_flush(void *args) -{ - struct wg_parser *parser = args; - - pthread_mutex_lock(&parser->mutex); - parser->flushing = false; - pthread_mutex_unlock(&parser->mutex); - - return S_OK; -} - static NTSTATUS wg_parser_get_next_read_offset(void *args) { struct wg_parser_get_next_read_offset_params *params = args; @@ -288,16 +259,9 @@ static NTSTATUS wg_parser_stream_get_event(void *args) pthread_mutex_lock(&parser->mutex); - while (!parser->flushing && stream->event.type == WG_PARSER_EVENT_NONE) + while (stream->event.type == WG_PARSER_EVENT_NONE) pthread_cond_wait(&stream->event_cond, &parser->mutex); - if (parser->flushing) - { - pthread_mutex_unlock(&parser->mutex); - GST_DEBUG("Filter is flushing.\n"); - return VFW_E_WRONG_STATE; - } - *params->event = stream->event; if (stream->event.type != WG_PARSER_EVENT_BUFFER) @@ -1593,7 +1557,6 @@ static NTSTATUS wg_parser_create(void *args) pthread_cond_init(&parser->init_cond, NULL); pthread_cond_init(&parser->read_cond, NULL); pthread_cond_init(&parser->read_done_cond, NULL); - parser->flushing = true; parser->init_gst = init_funcs[params->type]; parser->unlimited_buffering = params->unlimited_buffering; @@ -1630,9 +1593,6 @@ const unixlib_entry_t __wine_unix_call_funcs[] = X(wg_parser_connect), X(wg_parser_disconnect), - X(wg_parser_begin_flush), - X(wg_parser_end_flush), - X(wg_parser_get_next_read_offset), X(wg_parser_push_data), diff --git a/dlls/winegstreamer/wm_reader.c b/dlls/winegstreamer/wm_reader.c index f49d99071e0..6d2b9edbd09 100644 --- a/dlls/winegstreamer/wm_reader.c +++ b/dlls/winegstreamer/wm_reader.c @@ -1512,7 +1512,6 @@ static HRESULT init_stream(struct wm_reader *reader, QWORD file_size) wg_parser_stream_enable(stream->wg_stream, &stream->format); } - wg_parser_end_flush(reader->wg_parser); /* We probably discarded events because streams weren't enabled yet. * Now that they're all enabled seek back to the start again. */ wg_parser_stream_seek(reader->streams[0].wg_stream, 1.0, 0, 0, @@ -1836,11 +1835,7 @@ HRESULT wm_reader_get_stream_sample(struct wm_stream *stream, for (;;) { - if (!wg_parser_stream_get_event(wg_stream, &event)) - { - FIXME("Stream is flushing.\n"); - return E_NOTIMPL; - } + wg_parser_stream_get_event(wg_stream, &event); TRACE("Got event of type %#x for %s stream %p.\n", event.type, get_major_type_string(stream->format.major_type), stream);

1 0

Zebediah Figura : winegstreamer: Get rid of the queue_stream_event() helper.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: 0c59cfc14e33a5706c1873151780d5b2dcc536ca URL: https://source.winehq.org/git/wine.git/?a=commit;h=0c59cfc14e33a5706c187315… Author: Zebediah Figura <zfigura(a)codeweavers.com> Date: Wed Feb 23 15:46:49 2022 -0600 winegstreamer: Get rid of the queue_stream_event() helper. Signed-off-by: Zebediah Figura <zfigura(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winegstreamer/wg_parser.c | 76 +++++++++++++++++++----------------------- 1 file changed, 35 insertions(+), 41 deletions(-) diff --git a/dlls/winegstreamer/wg_parser.c b/dlls/winegstreamer/wg_parser.c index 3cdeeeb5374..e114a19a26b 100644 --- a/dlls/winegstreamer/wg_parser.c +++ b/dlls/winegstreamer/wg_parser.c @@ -415,43 +415,6 @@ static void no_more_pads_cb(GstElement *element, gpointer user) pthread_cond_signal(&parser->init_cond); } -static GstFlowReturn queue_stream_event(struct wg_parser_stream *stream, - const struct wg_parser_event *event, GstBuffer *buffer) -{ - struct wg_parser *parser = stream->parser; - - /* Unlike request_buffer_src() [q.v.], we need to watch for GStreamer - * flushes here. The difference is that we can be blocked by the streaming - * thread not running (or itself flushing on the DirectShow side). - * request_buffer_src() can only be blocked by the upstream source, and that - * is solved by flushing the upstream source. */ - - pthread_mutex_lock(&parser->mutex); - while (!stream->flushing && stream->event.type != WG_PARSER_EVENT_NONE) - pthread_cond_wait(&stream->event_empty_cond, &parser->mutex); - if (stream->flushing) - { - pthread_mutex_unlock(&parser->mutex); - GST_DEBUG("Filter is flushing; discarding event."); - return GST_FLOW_FLUSHING; - } - - assert(GST_IS_BUFFER(buffer)); - if (!gst_buffer_map(buffer, &stream->map_info, GST_MAP_READ)) - { - pthread_mutex_unlock(&parser->mutex); - GST_ERROR("Failed to map buffer.\n"); - return GST_FLOW_ERROR; - } - - stream->event = *event; - stream->buffer = buffer; - pthread_mutex_unlock(&parser->mutex); - pthread_cond_signal(&stream->event_cond); - GST_LOG("Event queued."); - return GST_FLOW_OK; -} - static gboolean sink_event_cb(GstPad *pad, GstObject *parent, GstEvent *event) { struct wg_parser_stream *stream = gst_pad_get_element_private(pad); @@ -550,8 +513,8 @@ static gboolean sink_event_cb(GstPad *pad, GstObject *parent, GstEvent *event) static GstFlowReturn sink_chain_cb(GstPad *pad, GstObject *parent, GstBuffer *buffer) { struct wg_parser_stream *stream = gst_pad_get_element_private(pad); + struct wg_parser *parser = stream->parser; struct wg_parser_event stream_event; - GstFlowReturn ret; GST_LOG("stream %p, buffer %p.", stream, buffer); @@ -576,10 +539,41 @@ static GstFlowReturn sink_chain_cb(GstPad *pad, GstObject *parent, GstBuffer *bu stream_event.u.buffer.delta = GST_BUFFER_FLAG_IS_SET(buffer, GST_BUFFER_FLAG_DELTA_UNIT); stream_event.u.buffer.size = gst_buffer_get_size(buffer); - /* Transfer our reference to the buffer to the stream object. */ - if ((ret = queue_stream_event(stream, &stream_event, buffer)) != GST_FLOW_OK) + /* Allow this buffer to be flushed by GStreamer. We are effectively + * implementing a queue object here. */ + + pthread_mutex_lock(&parser->mutex); + + while (!stream->flushing && stream->event.type != WG_PARSER_EVENT_NONE) + pthread_cond_wait(&stream->event_empty_cond, &parser->mutex); + if (stream->flushing) + { + pthread_mutex_unlock(&parser->mutex); + GST_DEBUG("Stream is flushing; discarding buffer."); gst_buffer_unref(buffer); - return ret; + return GST_FLOW_FLUSHING; + } + + if (!gst_buffer_map(buffer, &stream->map_info, GST_MAP_READ)) + { + pthread_mutex_unlock(&parser->mutex); + GST_ERROR("Failed to map buffer.\n"); + gst_buffer_unref(buffer); + return GST_FLOW_ERROR; + } + + stream->event = stream_event; + stream->buffer = buffer; + + pthread_mutex_unlock(&parser->mutex); + pthread_cond_signal(&stream->event_cond); + + /* The chain callback is given a reference to the buffer. Transfer that + * reference to the stream object, which will release it in + * wg_parser_stream_release_buffer(). */ + + GST_LOG("Buffer queued."); + return GST_FLOW_OK; } static gboolean sink_query_cb(GstPad *pad, GstObject *parent, GstQuery *query)

1 0

Jacek Caban : user32: Don't access cursoricon_object directly in CopyImage.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: 4f3f1c985adff0332fd6a34962da530922fd655d URL: https://source.winehq.org/git/wine.git/?a=commit;h=4f3f1c985adff0332fd6a349… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 24 01:27:24 2022 +0100 user32: Don't access cursoricon_object directly in CopyImage. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/cursoricon.c | 110 ++++++++++++++++++----------------------------- 1 file changed, 41 insertions(+), 69 deletions(-) diff --git a/dlls/user32/cursoricon.c b/dlls/user32/cursoricon.c index 509a5c3f4ed..d0aaceb14a0 100644 --- a/dlls/user32/cursoricon.c +++ b/dlls/user32/cursoricon.c @@ -72,37 +72,6 @@ static int get_display_bpp(void) return ret; } -static struct cursoricon_object *get_icon_ptr( HICON handle ) -{ - struct cursoricon_object *obj = get_user_handle_ptr( handle, NTUSER_OBJ_ICON ); - if (obj == OBJ_OTHER_PROCESS) - { - WARN( "icon handle %p from other process\n", handle ); - obj = NULL; - } - return obj; -} - -static struct cursoricon_frame *get_icon_frame( struct cursoricon_object *obj, int istep ) -{ - struct cursoricon_object *req_frame; - - if (!obj->is_ani) return &obj->frame; - if (!(req_frame = get_icon_ptr( obj->ani.frames[istep] ))) return 0; - return &req_frame->frame; -} - -static void release_icon_frame( struct cursoricon_object *obj, struct cursoricon_frame *frame ) -{ - if (obj->is_ani) - { - struct cursoricon_object *frameobj; - - frameobj = (struct cursoricon_object *) (((char *)frame) - FIELD_OFFSET(struct cursoricon_object, frame)); - release_user_handle_ptr( frameobj ); - } -} - static void free_icon_frame( struct cursoricon_frame *frame ) { if (frame->color) DeleteObject( frame->color ); @@ -2481,29 +2450,31 @@ HANDLE WINAPI CopyImage( HANDLE hnd, UINT type, INT desiredx, case IMAGE_ICON: case IMAGE_CURSOR: { - struct cursoricon_frame *frame; - struct cursoricon_object *icon; + ICONINFOEXW icon_info; int depth = (flags & LR_MONOCHROME) ? 1 : get_display_bpp(); - HICON resource_icon = NULL; - HINSTANCE module; + HICON resource_icon = 0; + LONG width, height; ICONINFO info; + HANDLE module; HICON res; - if (!(icon = get_icon_ptr( hnd ))) return 0; + icon_info.cbSize = sizeof(icon_info); + if (!GetIconInfoExW( hnd, &icon_info )) return 0; - if (icon->rsrc && (flags & LR_COPYFROMRESOURCE) && icon->module.Length && - !LdrGetDllHandle( NULL, 0, &icon->module, &module )) + if (icon_info.szModName[0] && (flags & LR_COPYFROMRESOURCE) && + (module = GetModuleHandleW( icon_info.szModName ))) { - resource_icon = CURSORICON_Load( module, icon->resname, desiredx, - desiredy, depth, !icon->is_icon, flags ); - release_user_handle_ptr( icon ); - if (!(icon = get_icon_ptr( resource_icon ))) - { - if (resource_icon) DestroyIcon( resource_icon ); - return 0; - } + const WCHAR *res = icon_info.szResName[0] ? icon_info.szResName + : MAKEINTRESOURCEW( icon_info.wResID ); + resource_icon = CURSORICON_Load( module, res, desiredx, desiredy, depth, + !icon_info.fIcon, flags ); + DeleteObject( icon_info.hbmColor ); + DeleteObject( icon_info.hbmMask ); + NtUserGetIconSize( resource_icon, 0, &width, &height ); + if (!GetIconInfoExW( resource_icon, &icon_info )) return 0; } - frame = get_icon_frame( icon, 0 ); + else NtUserGetIconSize( hnd, 0, &width, &height ); + height /= 2; if (flags & LR_DEFAULTSIZE) { @@ -2512,44 +2483,44 @@ HANDLE WINAPI CopyImage( HANDLE hnd, UINT type, INT desiredx, } else { - if (!desiredx) desiredx = frame->width; - if (!desiredy) desiredy = frame->height; + if (!desiredx) desiredx = width; + if (!desiredy) desiredy = height; } - info.fIcon = icon->is_icon; - info.xHotspot = frame->hotspot.x; - info.yHotspot = frame->hotspot.y; + info.fIcon = icon_info.fIcon; + info.xHotspot = icon_info.xHotspot; + info.yHotspot = icon_info.yHotspot; - if (desiredx == frame->width && desiredy == frame->height) + if (desiredx == width && desiredy == height) { - info.hbmColor = frame->color; - info.hbmMask = frame->mask; + info.hbmColor = icon_info.hbmColor; + info.hbmMask = icon_info.hbmMask; res = CreateIconIndirect( &info ); } else { - if (frame->color) + if (icon_info.hbmColor) { if (!(info.hbmColor = create_color_bitmap( desiredx, desiredy ))) { - release_icon_frame( icon, frame ); - release_user_handle_ptr( icon ); + DeleteObject( icon_info.hbmColor ); + DeleteObject( icon_info.hbmMask ); if (resource_icon) DestroyIcon( resource_icon ); return 0; } - stretch_bitmap( info.hbmColor, frame->color, desiredx, desiredy, - frame->width, frame->height ); + stretch_bitmap( info.hbmColor, icon_info.hbmColor, desiredx, desiredy, + width, height ); if (!(info.hbmMask = CreateBitmap( desiredx, desiredy, 1, 1, NULL ))) { + DeleteObject( icon_info.hbmColor ); + DeleteObject( icon_info.hbmMask ); DeleteObject( info.hbmColor ); - release_icon_frame( icon, frame ); - release_user_handle_ptr( icon ); if (resource_icon) DestroyIcon( resource_icon ); return 0; } - stretch_bitmap( info.hbmMask, frame->mask, desiredx, desiredy, - frame->width, frame->height ); + stretch_bitmap( info.hbmMask, icon_info.hbmMask, desiredx, desiredy, + width, height ); } else { @@ -2557,12 +2528,13 @@ HANDLE WINAPI CopyImage( HANDLE hnd, UINT type, INT desiredx, if (!(info.hbmMask = CreateBitmap( desiredx, desiredy * 2, 1, 1, NULL ))) { - release_user_handle_ptr( icon ); + DeleteObject( icon_info.hbmColor ); + DeleteObject( icon_info.hbmMask ); if (resource_icon) DestroyIcon( resource_icon ); return 0; } - stretch_bitmap( info.hbmMask, frame->mask, desiredx, desiredy * 2, - frame->width, frame->height * 2 ); + stretch_bitmap( info.hbmMask, icon_info.hbmMask, desiredx, desiredy * 2, + width, height * 2 ); } res = CreateIconIndirect( &info ); @@ -2571,8 +2543,8 @@ HANDLE WINAPI CopyImage( HANDLE hnd, UINT type, INT desiredx, DeleteObject( info.hbmMask ); } - release_icon_frame( icon, frame ); - release_user_handle_ptr( icon ); + DeleteObject( icon_info.hbmColor ); + DeleteObject( icon_info.hbmMask ); if (res && (flags & LR_COPYDELETEORG)) DestroyIcon( hnd ); if (resource_icon) DestroyIcon( resource_icon );

1 0

Jacek Caban : win32u: Move get_icon_param and set_icon_param from user32.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: 439fb2519091c282094d484ecdae98959fe26142 URL: https://source.winehq.org/git/wine.git/?a=commit;h=439fb2519091c282094d484e… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 24 01:27:36 2022 +0100 win32u: Move get_icon_param and set_icon_param from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/cursoricon.c | 23 ++----------------- dlls/win32u/cursoricon.c | 53 ++++++++++++++++++++++++++++++++++++++++++++ dlls/win32u/ntuser_private.h | 24 -------------------- dlls/win32u/sysparams.c | 4 ++++ dlls/win32u/win32u_private.h | 2 ++ include/ntuser.h | 2 ++ 6 files changed, 63 insertions(+), 45 deletions(-) diff --git a/dlls/user32/cursoricon.c b/dlls/user32/cursoricon.c index d0aaceb14a0..cab9fbde32e 100644 --- a/dlls/user32/cursoricon.c +++ b/dlls/user32/cursoricon.c @@ -81,31 +81,12 @@ static void free_icon_frame( struct cursoricon_frame *frame ) ULONG_PTR get_icon_param( HICON handle ) { - ULONG_PTR ret = 0; - struct cursoricon_object *obj = get_user_handle_ptr( handle, NTUSER_OBJ_ICON ); - - if (obj == OBJ_OTHER_PROCESS) WARN( "icon handle %p from other process\n", handle ); - else if (obj) - { - ret = obj->param; - release_user_handle_ptr( obj ); - } - return ret; + return NtUserCallOneParam( HandleToUlong(handle), NtUserGetIconParam ); } ULONG_PTR set_icon_param( HICON handle, ULONG_PTR param ) { - ULONG_PTR ret = 0; - struct cursoricon_object *obj = get_user_handle_ptr( handle, NTUSER_OBJ_ICON ); - - if (obj == OBJ_OTHER_PROCESS) WARN( "icon handle %p from other process\n", handle ); - else if (obj) - { - ret = obj->param; - obj->param = param; - release_user_handle_ptr( obj ); - } - return ret; + return NtUserCallTwoParam( HandleToUlong(handle), param, NtUserSetIconParam ); } diff --git a/dlls/win32u/cursoricon.c b/dlls/win32u/cursoricon.c index e2c121d6c1e..0249eb3da50 100644 --- a/dlls/win32u/cursoricon.c +++ b/dlls/win32u/cursoricon.c @@ -37,6 +37,30 @@ WINE_DEFAULT_DEBUG_CHANNEL(cursor); WINE_DECLARE_DEBUG_CHANNEL(icon); +struct cursoricon_object +{ + struct user_object obj; /* object header */ + struct list entry; /* entry in shared icons list */ + ULONG_PTR param; /* opaque param used by 16-bit code */ + UNICODE_STRING module; /* module for icons loaded from resources */ + WCHAR *resname; /* resource name for icons loaded from resources */ + HRSRC rsrc; /* resource for shared icons */ + BOOL is_shared; /* whether this object is shared */ + BOOL is_icon; /* whether icon or cursor */ + BOOL is_ani; /* whether this object is a static cursor or an animated cursor */ + UINT delay; /* delay between this frame and the next (in jiffies) */ + union + { + struct cursoricon_frame frame; /* frame-specific icon data */ + struct + { + UINT num_frames; /* number of frames in the icon/cursor */ + UINT num_steps; /* number of sequence steps in the icon/cursor */ + HICON *frames; /* list of animated cursor frames */ + } ani; + }; +}; + static struct list icon_cache = LIST_INIT( icon_cache ); static struct cursoricon_object *get_icon_ptr( HICON handle ) @@ -733,3 +757,32 @@ failed: release_user_handle_ptr( obj ); return result; } + +ULONG_PTR get_icon_param( HICON handle ) +{ + ULONG_PTR ret = 0; + struct cursoricon_object *obj = get_user_handle_ptr( handle, NTUSER_OBJ_ICON ); + + if (obj == OBJ_OTHER_PROCESS) WARN( "icon handle %p from other process\n", handle ); + else if (obj) + { + ret = obj->param; + release_user_handle_ptr( obj ); + } + return ret; +} + +ULONG_PTR set_icon_param( HICON handle, ULONG_PTR param ) +{ + ULONG_PTR ret = 0; + struct cursoricon_object *obj = get_user_handle_ptr( handle, NTUSER_OBJ_ICON ); + + if (obj == OBJ_OTHER_PROCESS) WARN( "icon handle %p from other process\n", handle ); + else if (obj) + { + ret = obj->param; + obj->param = param; + release_user_handle_ptr( obj ); + } + return ret; +} diff --git a/dlls/win32u/ntuser_private.h b/dlls/win32u/ntuser_private.h index db1a08ff102..f79930ba229 100644 --- a/dlls/win32u/ntuser_private.h +++ b/dlls/win32u/ntuser_private.h @@ -125,30 +125,6 @@ struct user_key_state_info BYTE state[256]; /* State for each key */ }; -struct cursoricon_object -{ - struct user_object obj; /* object header */ - struct list entry; /* entry in shared icons list */ - ULONG_PTR param; /* opaque param used by 16-bit code */ - UNICODE_STRING module; /* module for icons loaded from resources */ - LPWSTR resname; /* resource name for icons loaded from resources */ - HRSRC rsrc; /* resource for shared icons */ - BOOL is_shared; /* whether this object is shared */ - BOOL is_icon; /* whether icon or cursor */ - BOOL is_ani; /* whether this object is a static cursor or an animated cursor */ - UINT delay; /* delay between this frame and the next (in jiffies) */ - union - { - struct cursoricon_frame frame; /* frame-specific icon data */ - struct - { - UINT num_frames; /* number of frames in the icon/cursor */ - UINT num_steps; /* number of sequence steps in the icon/cursor */ - HICON *frames; /* list of animated cursor frames */ - } ani; - }; -}; - /* cursoricon.c */ HICON alloc_cursoricon_handle( BOOL is_icon ) DECLSPEC_HIDDEN; diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index 70d310646ef..4eaaceb2892 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -4558,6 +4558,8 @@ ULONG_PTR WINAPI NtUserCallOneParam( ULONG_PTR arg, ULONG code ) return get_clip_cursor( (RECT *)arg ); case NtUserGetCursorPos: return get_cursor_pos( (POINT *)arg ); + case NtUserGetIconParam: + return get_icon_param( UlongToHandle(arg) ); case NtUserGetSysColor: return get_sys_color( arg ); case NtUserRealizePalette: @@ -4616,6 +4618,8 @@ ULONG_PTR WINAPI NtUserCallTwoParam( ULONG_PTR arg1, ULONG_PTR arg2, ULONG code return mirror_window_region( UlongToHandle(arg1), UlongToHandle(arg2) ); case NtUserMonitorFromRect: return HandleToUlong( monitor_from_rect( (const RECT *)arg1, arg2, get_thread_dpi() )); + case NtUserSetIconParam: + return set_icon_param( UlongToHandle(arg1), arg2 ); case NtUserUnhookWindowsHook: return unhook_windows_hook( arg1, (HOOKPROC)arg2 ); /* temporary exports */ diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index 9ff6d3143db..64712a66e12 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -247,6 +247,8 @@ struct unix_funcs /* cursoricon.c */ extern HICON alloc_cursoricon_handle( BOOL is_icon ) DECLSPEC_HIDDEN; extern BOOL get_clip_cursor( RECT *rect ) DECLSPEC_HIDDEN; +extern ULONG_PTR get_icon_param( HICON handle ) DECLSPEC_HIDDEN; +extern ULONG_PTR set_icon_param( HICON handle, ULONG_PTR param ) DECLSPEC_HIDDEN; /* hook.c */ extern BOOL unhook_windows_hook( INT id, HOOKPROC proc ) DECLSPEC_HIDDEN; diff --git a/include/ntuser.h b/include/ntuser.h index ecefd2d68cb..a4184c944d4 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -80,6 +80,7 @@ enum NtUserCreateCursorIcon, NtUserGetClipCursor, NtUserGetCursorPos, + NtUserGetIconParam, NtUserGetPrimaryMonitorRect, NtUserGetSysColor, NtUserGetSysColorBrush, @@ -104,6 +105,7 @@ enum NtUserGetSystemMetricsForDpi, NtUserMirrorRgn, NtUserMonitorFromRect, + NtUserSetIconParam, NtUserUnhookWindowsHook, /* temporary exports */ NtUserAllocHandle,

1 0

Jacek Caban : win32u: Move NtUserGetCursorInfo implementation from user32.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: 47ca0f49c7fe3d439993a8c7e885bfea4b5bae40 URL: https://source.winehq.org/git/wine.git/?a=commit;h=47ca0f49c7fe3d439993a8c7… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 24 01:28:18 2022 +0100 win32u: Move NtUserGetCursorInfo implementation from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/input.c | 24 ------------------------ dlls/user32/user32.spec | 2 +- dlls/win32u/gdiobj.c | 1 + dlls/win32u/input.c | 23 +++++++++++++++++++++++ dlls/win32u/win32u.spec | 2 +- dlls/win32u/win32u_private.h | 1 + dlls/win32u/wrappers.c | 6 ++++++ include/ntuser.h | 1 + 8 files changed, 34 insertions(+), 26 deletions(-) diff --git a/dlls/user32/input.c b/dlls/user32/input.c index 781896fd16b..6cf60abeae2 100644 --- a/dlls/user32/input.c +++ b/dlls/user32/input.c @@ -307,30 +307,6 @@ BOOL WINAPI DECLSPEC_HOTPATCH GetCursorPos( POINT *pt ) } -/*********************************************************************** - * GetCursorInfo (USER32.@) - */ -BOOL WINAPI GetCursorInfo( PCURSORINFO pci ) -{ - BOOL ret; - - if (!pci) return FALSE; - - SERVER_START_REQ( get_thread_input ) - { - req->tid = 0; - if ((ret = !wine_server_call( req ))) - { - pci->hCursor = wine_server_ptr_handle( reply->cursor ); - pci->flags = (reply->show_count >= 0) ? CURSOR_SHOWING : 0; - } - } - SERVER_END_REQ; - GetCursorPos(&pci->ptScreenPos); - return ret; -} - - /********************************************************************** * SetCapture (USER32.@) */ diff --git a/dlls/user32/user32.spec b/dlls/user32/user32.spec index 5204852bef5..40044023bf4 100644 --- a/dlls/user32/user32.spec +++ b/dlls/user32/user32.spec @@ -284,7 +284,7 @@ @ stdcall GetCurrentInputMessageSource(ptr) @ stdcall GetCursor() NtUserGetCursor @ stdcall GetCursorFrameInfo(long long long ptr ptr) -@ stdcall GetCursorInfo(ptr) +@ stdcall GetCursorInfo(ptr) NtUserGetCursorInfo @ stdcall GetCursorPos(ptr) @ stdcall GetDC(long) @ stdcall GetDCEx(long long long) diff --git a/dlls/win32u/gdiobj.c b/dlls/win32u/gdiobj.c index 2ac40d71e3c..1d54865bf46 100644 --- a/dlls/win32u/gdiobj.c +++ b/dlls/win32u/gdiobj.c @@ -1162,6 +1162,7 @@ static struct unix_funcs unix_funcs = NtUserEnumDisplayMonitors, NtUserEnumDisplaySettings, NtUserGetAsyncKeyState, + NtUserGetCursorInfo, NtUserGetDisplayConfigBufferSizes, NtUserGetIconInfo, NtUserGetKeyNameText, diff --git a/dlls/win32u/input.c b/dlls/win32u/input.c index f8f4d41ad2c..72af32fd471 100644 --- a/dlls/win32u/input.c +++ b/dlls/win32u/input.c @@ -131,6 +131,29 @@ BOOL get_cursor_pos( POINT *pt ) return ret; } +/*********************************************************************** + * NtUserGetCursorInfo (win32u.@) + */ +BOOL WINAPI NtUserGetCursorInfo( CURSORINFO *info ) +{ + BOOL ret; + + if (!info) return FALSE; + + SERVER_START_REQ( get_thread_input ) + { + req->tid = 0; + if ((ret = !wine_server_call( req ))) + { + info->hCursor = wine_server_ptr_handle( reply->cursor ); + info->flags = reply->show_count >= 0 ? CURSOR_SHOWING : 0; + } + } + SERVER_END_REQ; + get_cursor_pos( &info->ptScreenPos ); + return ret; +} + static void check_for_events( UINT flags ) { if (user_driver->pMsgWaitForMultipleObjectsEx( 0, NULL, 0, flags, 0 ) == WAIT_TIMEOUT) diff --git a/dlls/win32u/win32u.spec b/dlls/win32u/win32u.spec index 4a743c7d37e..c1dca3d425c 100644 --- a/dlls/win32u/win32u.spec +++ b/dlls/win32u/win32u.spec @@ -914,7 +914,7 @@ @ stub NtUserGetCurrentInputMessageSource @ stdcall -syscall NtUserGetCursor() @ stdcall -syscall NtUserGetCursorFrameInfo(long long ptr ptr) -@ stub NtUserGetCursorInfo +@ stdcall NtUserGetCursorInfo(ptr) @ stub NtUserGetDC @ stub NtUserGetDCEx @ stub NtUserGetDManipHookInitFunction diff --git a/dlls/win32u/win32u_private.h b/dlls/win32u/win32u_private.h index 9b94387be79..002524a18f4 100644 --- a/dlls/win32u/win32u_private.h +++ b/dlls/win32u/win32u_private.h @@ -199,6 +199,7 @@ struct unix_funcs BOOL (WINAPI *pNtUserEnumDisplaySettings)( UNICODE_STRING *device, DWORD mode, DEVMODEW *dev_mode, DWORD flags ); SHORT (WINAPI *pNtUserGetAsyncKeyState)( INT key ); + BOOL (WINAPI *pNtUserGetCursorInfo)( CURSORINFO *info ); LONG (WINAPI *pNtUserGetDisplayConfigBufferSizes)( UINT32 flags, UINT32 *num_path_info, UINT32 *num_mode_info ); BOOL (WINAPI *pNtUserGetIconInfo)( HICON icon, ICONINFO *info, UNICODE_STRING *module, diff --git a/dlls/win32u/wrappers.c b/dlls/win32u/wrappers.c index 8d57fc20036..ca69422f086 100644 --- a/dlls/win32u/wrappers.c +++ b/dlls/win32u/wrappers.c @@ -777,6 +777,12 @@ SHORT WINAPI NtUserGetAsyncKeyState( INT key ) return unix_funcs->pNtUserGetAsyncKeyState( key ); } +BOOL WINAPI NtUserGetCursorInfo( CURSORINFO *info ) +{ + if (!unix_funcs) return FALSE; + return unix_funcs->pNtUserGetCursorInfo( info ); +} + LONG WINAPI NtUserGetDisplayConfigBufferSizes( UINT32 flags, UINT32 *num_path_info, UINT32 *num_mode_info ) { diff --git a/include/ntuser.h b/include/ntuser.h index fdc33ce7e70..8730c739b11 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -205,6 +205,7 @@ HWND WINAPI NtUserGetClipboardViewer(void); HCURSOR WINAPI NtUserGetCursor(void); HCURSOR WINAPI NtUserGetCursorFrameInfo( HCURSOR hCursor, DWORD istep, DWORD *rate_jiffies, DWORD *num_steps ); +BOOL WINAPI NtUserGetCursorInfo( CURSORINFO *info ); LONG WINAPI NtUserGetDisplayConfigBufferSizes( UINT32 flags, UINT32 *num_path_info, UINT32 *num_mode_info ); UINT WINAPI NtUserGetDoubleClickTime(void);

1 0

Jacek Caban : user32: Use NtUserGetGUIThreadInfo in GetActiveWindow.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: fa2f52578bf1d9e386fa47c89e90671f9f9a8e6c URL: https://source.winehq.org/git/wine.git/?a=commit;h=fa2f52578bf1d9e386fa47c8… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 24 01:28:38 2022 +0100 user32: Use NtUserGetGUIThreadInfo in GetActiveWindow. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/focus.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/dlls/user32/focus.c b/dlls/user32/focus.c index d699dd7eecd..42787bac60a 100644 --- a/dlls/user32/focus.c +++ b/dlls/user32/focus.c @@ -342,15 +342,9 @@ BOOL WINAPI SetForegroundWindow( HWND hwnd ) */ HWND WINAPI GetActiveWindow(void) { - HWND ret = 0; - - SERVER_START_REQ( get_thread_input ) - { - req->tid = GetCurrentThreadId(); - if (!wine_server_call_err( req )) ret = wine_server_ptr_handle( reply->active ); - } - SERVER_END_REQ; - return ret; + GUITHREADINFO info; + info.cbSize = sizeof(info); + return NtUserGetGUIThreadInfo( GetCurrentThreadId(), &info ) ? info.hwndActive : 0; }

1 0

Jacek Caban : user32: Use NtUserGetGUIThreadInfo in GetFocus.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: ab45fed26af328aa32708ba1cbfe6e463af59073 URL: https://source.winehq.org/git/wine.git/?a=commit;h=ab45fed26af328aa32708ba1… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 24 01:28:44 2022 +0100 user32: Use NtUserGetGUIThreadInfo in GetFocus. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/focus.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/dlls/user32/focus.c b/dlls/user32/focus.c index 42787bac60a..f0b5ea2552e 100644 --- a/dlls/user32/focus.c +++ b/dlls/user32/focus.c @@ -353,15 +353,9 @@ HWND WINAPI GetActiveWindow(void) */ HWND WINAPI GetFocus(void) { - HWND ret = 0; - - SERVER_START_REQ( get_thread_input ) - { - req->tid = GetCurrentThreadId(); - if (!wine_server_call_err( req )) ret = wine_server_ptr_handle( reply->focus ); - } - SERVER_END_REQ; - return ret; + GUITHREADINFO info; + info.cbSize = sizeof(info); + return NtUserGetGUIThreadInfo( GetCurrentThreadId(), &info ) ? info.hwndFocus : 0; }

1 0

Jacek Caban : win32u: Move NtUserRegisterHotKey implementation from user32.
by Alexandre Julliard Feb. 24, 2022

Feb. 24, 2022

Module: wine Branch: master Commit: e560357df12fc42957504ffb5b6cbe13cded197e URL: https://source.winehq.org/git/wine.git/?a=commit;h=e560357df12fc42957504ffb… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 24 01:27:57 2022 +0100 win32u: Move NtUserRegisterHotKey implementation from user32. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/user32/driver.c | 25 ++------------------ dlls/user32/input.c | 35 --------------------------- dlls/user32/user32.spec | 2 +- dlls/user32/win.h | 3 --- dlls/win32u/driver.c | 6 +++++ dlls/win32u/gdiobj.c | 1 + dlls/win32u/input.c | 35 +++++++++++++++++++++++++++ dlls/win32u/ntuser_private.h | 3 +++ dlls/win32u/win32u.spec | 2 +- dlls/win32u/win32u_private.h | 15 +++++++++--- dlls/win32u/window.c | 56 ++++++++++++++++++++++++++++++++++++++++++++ dlls/win32u/wrappers.c | 6 +++++ include/ntuser.h | 1 + 13 files changed, 124 insertions(+), 66 deletions(-) Diff: https://source.winehq.org/git/wine.git/?a=commitdiff;h=e560357df12fc4295750…

1 0