mirror of
https://github.com/mpv-player/mpv
synced 2025-03-09 15:47:44 +00:00
24357cb7b5
The source of many geometry woes. There's some loosely related toplevel
things that should be cleaned up/fixed. First of all,
VO_EVENT_LIVE_RESIZING is actually completely useless. It might have
been useful sometime in the past, but there's no point. It doesn't
"speed up" resizing in any way and appears to be originally for cocoa.
Just remove it.
Way back in the day, toplevel_width/height was added as a workaround for
when we got uncoorperative (i.e. wrong) width/height coordinates from
the compositor in this event. Basically it could happen due to numerous
reasons but a lack of atomic commits was part of the reason and also
mpv's geometry handling then was a lot rougher. We *shouldn't* need this
workaround anymore. The width/height values are only used exactly when
we need them. If mpv sets geometry on its own, it should still be the
right dimensions.
Related to the above, mpv never actually propertly handled the case
where width or height was equal to 0. According to the xdg-shell spec,
"If the width or height arguments are zero, it means the client should
decided its own window dimension." An example of a compositor doing this
is weston. It's, unsurprisingly, broken. Getting out of fullscreen or a
maximized state does not restore the old window size like it should. The
right way to handle this is to just return near the end of the function
if we have a 0 for either argument and before any geometry is set
(wl->geometry's width or height can never be zero). Luckily, state
changes are already being detected so they just trigger the goto when
needed.
Finally,
|
||
|---|---|---|
| .. | ||
| cocoa | ||
| d3d11 | ||
| gpu | ||
| hwdec | ||
| mac | ||
| opengl | ||
| placebo | ||
| vulkan | ||
| win32 | ||
| android_common.c | ||
| android_common.h | ||
| aspect.c | ||
| aspect.h | ||
| bitmap_packer.c | ||
| bitmap_packer.h | ||
| cocoa_cb_common.swift | ||
| cocoa_common.h | ||
| cocoa_common.m | ||
| dither.c | ||
| dither.h | ||
| dr_helper.c | ||
| dr_helper.h | ||
| drm_atomic.c | ||
| drm_atomic.h | ||
| drm_common.c | ||
| drm_common.h | ||
| drm_prime.c | ||
| drm_prime.h | ||
| filter_kernels.c | ||
| filter_kernels.h | ||
| libmpv_sw.c | ||
| libmpv.h | ||
| vo_caca.c | ||
| vo_direct3d.c | ||
| vo_drm.c | ||
| vo_gpu.c | ||
| vo_image.c | ||
| vo_lavc.c | ||
| vo_libmpv.c | ||
| vo_mediacodec_embed.c | ||
| vo_null.c | ||
| vo_rpi.c | ||
| vo_sdl.c | ||
| vo_sixel.c | ||
| vo_tct.c | ||
| vo_vaapi.c | ||
| vo_vdpau.c | ||
| vo_wlshm.c | ||
| vo_x11.c | ||
| vo_xv.c | ||
| vo.c | ||
| vo.h | ||
| w32_common.c | ||
| w32_common.h | ||
| wayland_common.c | ||
| wayland_common.h | ||
| win_state.c | ||
| win_state.h | ||
| x11_common.c | ||
| x11_common.h | ||