[FFmpeg-devel] [PR] fftools/ffplay: enforce minimum window size (PR #21555)

[FgoDt via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>]

PR #21555 opened by FgoDt
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21555
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21555.patch

Ensure the playback window is not created smaller than 128 pixels
by introducing SDL_MIN_WINDOW_SIZE constant. Smaller window sizes
may cause window controls to be hidden or unusable.


>From 6ae031192ee0a583e2503d6d1340de37f6500988 Mon Sep 17 00:00:00 2001
From: zhao li <fgodtdev@gmail.com>
Date: Fri, 23 Jan 2026 16:03:11 +0800
Subject: [PATCH] fftools/ffplay: enforce minimum window size

Ensure the playback window is not created smaller than 128 pixels
by introducing SDL_MIN_WINDOW_SIZE constant. Smaller window sizes
may cause window controls to be hidden or unusable.
---
 fftools/ffplay.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/fftools/ffplay.c b/fftools/ffplay.c
index fe9aec05ab..1de586dd0a 100644
--- a/fftools/ffplay.c
+++ b/fftools/ffplay.c
@@ -75,6 +75,9 @@ const int program_birth_year = 2003;
 /* Step size for volume control in dB */
 #define SDL_VOLUME_STEP (0.75)
 
+/* Minimum window size; smaller sizes may cause window controls to be hidden */
+#define SDL_MIN_WINDOW_SIZE 128
+
 /* no AV sync correction is done if below the minimum AV sync threshold */
 #define AV_SYNC_THRESHOLD_MIN 0.04
 /* AV sync correction is done if above the maximum AV sync threshold */
@@ -1394,6 +1397,9 @@ static int video_open(VideoState *is)
     w = screen_width ? screen_width : default_width;
     h = screen_height ? screen_height : default_height;
 
+    w = FFMAX(w, SDL_MIN_WINDOW_SIZE);
+    h = FFMAX(h, SDL_MIN_WINDOW_SIZE);
+
     if (!window_title)
         window_title = input_filename;
     SDL_SetWindowTitle(window, window_title);
-- 
2.52.0

_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org