Skip to content

Comments

m_option: fix x:y mode format when printing geometry#17443

Merged
kasper93 merged 1 commit intompv-player:masterfrom
zenodity:fix-xy-mode-print-geometry
Feb 22, 2026
Merged

m_option: fix x:y mode format when printing geometry#17443
kasper93 merged 1 commit intompv-player:masterfrom
zenodity:fix-xy-mode-print-geometry

Conversation

@zenodity
Copy link
Contributor

@zenodity zenodity commented Feb 22, 2026

Previously, x:y mode geometry strings were incorrectly printed using the WxH+x+y format.

Fixes #17435

Tested the following:

mpv "av://lavfi:testsrc=size=1280x720:rate=1" --video-crop="150:100"
● Video --vid=1 (wrapped_avframe 1280x720 1 fps)
VO: [gpu-next] 1280x720 rgb24
video-crop: 150:100

mpv "av://lavfi:testsrc=size=360x180:rate=1" --video-crop="+150+100"
● Video --vid=1 (wrapped_avframe 360x180 1 fps)
VO: [gpu-next] 360x180 rgb24
video-crop: 0x0+150+100

mpv "av://lavfi:testsrc=size=1280x720:rate=1" --video-crop="50x10+150+100"
● Video --vid=1 (wrapped_avframe 1280x720 1 fps)
VO: [gpu-next] 1280x720 rgb24
video-crop: 50x10+150+100

mpv "av://lavfi:testsrc=size=1280x720:rate=1" --video-crop="50%:50%"
● Video --vid=1 (wrapped_avframe 1280x720 1 fps)
VO: [gpu-next] 1280x720 rgb24
video-crop: 50%:50%

Previously, x:y mode geometry strings were incorrectly printed
using the WxH+x+y format.
Copy link
Member

@kasper93 kasper93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@kasper93 kasper93 merged commit f439c7f into mpv-player:master Feb 22, 2026
27 of 29 checks passed
@zenodity zenodity deleted the fix-xy-mode-print-geometry branch February 23, 2026 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Printing video-crop and geometry "x:y" format value incorrectly returns "+x+y" format

2 participants