videolan
/
vlc
mirror of https://code.videolan.org/videolan/vlc
1
Fork 0
Code Releases Activity

display: remove redundant VOUT_DISPLAY_CHANGE_DISPLAY_FILLED 

If the display filling mode of the video changed, the video placement has changed as well.
Display modules don't actually care about the aspect ratio, just where they're supposed to stretch the video.
This commit is contained in:
Steve Lhomme 2024-04-10 07:29:05 +02:00
parent 0c10d95148
commit 65b813712c
21 changed files with 1 additions and 37 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
include/vlc_vout_display.h
View File

@ -146,15 +146,6 @@ enum vout_display_query {
*/
VOUT_DISPLAY_CHANGE_DISPLAY_SIZE,
/**
* Notifies a change of the display fitting mode by the user.
*
* \retval VLC_SUCCESS if the display handled the change
* \retval VLC_EGENERIC if a \ref vlc_display_operations::reset_pictures
* request is necessary
*/
VOUT_DISPLAY_CHANGE_DISPLAY_FILLED,
/**
* Notifies a change of the sample aspect ratio.
*
@ -294,7 +285,6 @@ struct vlc_display_operations
* Reset the picture format handled by the module.
* This occurs after a
* \ref VOUT_DISPLAY_CHANGE_DISPLAY_SIZE,
* \ref VOUT_DISPLAY_CHANGE_DISPLAY_FILLED,
* \ref VOUT_DISPLAY_CHANGE_SOURCE_ASPECT,
* \ref VOUT_DISPLAY_CHANGE_SOURCE_CROP or
* \ref VOUT_DISPLAY_CHANGE_SOURCE_PLACE

2
modules/video_output/android/display.c
View File

@ -77,7 +77,6 @@ static int subpicture_Control(vout_display_t *vd, int query)
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
vlc_gl_Resize(sub->gl, vd->cfg->display.width, vd->cfg->display.height);
// fallthrough
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:
{
sub->place_changed = true;
@ -422,7 +421,6 @@ static int Control(vout_display_t *vd, int query)
vd->cfg->display.height);
return VLC_SUCCESS;
}
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:
return VLC_SUCCESS;
default:

1
modules/video_output/apple/VLCSampleBufferDisplay.m
View File

@ -599,7 +599,6 @@ static int Control (vout_display_t *vd, int query)
switch (query)
{
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/caca.c
View File

@ -187,7 +187,6 @@ static int Control(vout_display_t *vd, int query)
sys->dither = NULL;
/* fall through */
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:
return VLC_SUCCESS;

1
modules/video_output/caopengllayer.m
View File

@ -554,7 +554,6 @@ static int Control (vout_display_t *vd, int query)
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
return VLC_SUCCESS;
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/decklink.cpp
View File

@ -765,7 +765,6 @@ static int ControlVideo(vout_display_t *vd, int query)
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/drm/display.c
View File

@ -77,7 +77,6 @@ static int Control(vout_display_t *vd, int query)
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/flaschen.c
View File

@ -233,7 +233,6 @@ static int Control(vout_display_t *vd, int query)
{
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/kva.c
View File

@ -406,7 +406,6 @@ static int Control( vout_display_t *vd, int query )
return VLC_SUCCESS;
}
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:
{

1
modules/video_output/libplacebo/display.c
View File

@ -546,7 +546,6 @@ static int Control(vout_display_t *vd, int query)
*/
}
return VLC_SUCCESS;
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/macosx.m
View File

@ -352,7 +352,6 @@ static int Control (vout_display_t *vd, int query)
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
return VLC_SUCCESS;
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/opengl/display.c
View File

@ -324,7 +324,6 @@ static int Control (vout_display_t *vd, int query)
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
vlc_gl_Resize (sys->gl, vd->cfg->display.width, vd->cfg->display.height);
// fallthrough
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/splitter.c
View File

@ -102,7 +102,6 @@ static int vlc_vidsplit_Control(vout_display_t *vd, int query)
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/vdummy.c
View File

@ -139,7 +139,6 @@ static int Control(vout_display_t *vd, int query)
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/vmem.c
View File

@ -252,7 +252,6 @@ static int Control(vout_display_t *vd, int query)
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/wayland/shm.c
View File

@ -180,7 +180,6 @@ static int Control(vout_display_t *vd, int query)
switch (query)
{
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/win32/common.c
View File

@ -121,7 +121,6 @@ void CommonControl(vout_display_t *vd, display_win32_area_t *area, int query)
}
#endif /* WINAPI_PARTITION_DESKTOP */
// fallthrough
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/xcb/render.c
View File

@ -394,7 +394,6 @@ static int Control(vout_display_t *vd, int query)
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/xcb/x11.c
View File

@ -222,7 +222,6 @@ static int Control(vout_display_t *vd, int query)
xcb_configure_window(sys->conn, sys->window, mask, values);
}
/* fall through */
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

1
modules/video_output/yuv.c
View File

@ -253,7 +253,6 @@ static int Control(vout_display_t *vd, int query)
switch (query) {
case VOUT_DISPLAY_CHANGE_DISPLAY_SIZE:
case VOUT_DISPLAY_CHANGE_DISPLAY_FILLED:
case VOUT_DISPLAY_CHANGE_SOURCE_ASPECT:
case VOUT_DISPLAY_CHANGE_SOURCE_CROP:
case VOUT_DISPLAY_CHANGE_SOURCE_PLACE:

8
src/video_output/display.c
View File

@ -623,18 +623,12 @@ void vout_SetDisplayFitting(vout_display_t *vd, enum vlc_video_fitting fit)
osys->cfg.display.fitting = fit;
bool place_changed = PlaceVideoInDisplay(osys);
int res1 = vout_display_Control(vd, VOUT_DISPLAY_CHANGE_DISPLAY_FILLED);
if (place_changed)
{
int res2 = vout_display_Control(vd, VOUT_DISPLAY_CHANGE_SOURCE_PLACE);
if (res2 != VLC_SUCCESS)
res1 = res2;
vout_display_Reset(vd);
}
if (res1 != VLC_SUCCESS)
vout_display_Reset(vd);
}
void vout_SetDisplayZoom(vout_display_t *vd, unsigned num, unsigned den)