From 3e89e061c25e32195677a2c7a239b512cdd3cca9 Mon Sep 17 00:00:00 2001 From: wm4 Date: Mon, 23 Jan 2017 15:28:12 +0100 Subject: sd_lavc: remove old broken heuristic This core of this heuristic was once copied from MPlayer's spudec.c. I think it was meant for the case when the resolution field was missing or so. I couldn't find a file for which this actually does something. On the other hand, there are samples which actually have a smaller resolution than 720x576, and which are broken by this old hack. For subtitles that set no resolution (I'm not sure which codec/container that would be), there's still the fallback on video resolution. Just get rid of this hack. Also cleanup a bit. SD_CTRL_GET_RESOLUTION hasn't been used since DVD menu removal. get_resolution() is left with 1 call site, and would be quite awkward to keep, so un-inline it. --- sub/dec_sub.h | 1 - 1 file changed, 1 deletion(-) (limited to 'sub/dec_sub.h') diff --git a/sub/dec_sub.h b/sub/dec_sub.h index 341966a430..bdf6e35492 100644 --- a/sub/dec_sub.h +++ b/sub/dec_sub.h @@ -16,7 +16,6 @@ struct sd; enum sd_ctrl { SD_CTRL_SUB_STEP, SD_CTRL_SET_VIDEO_PARAMS, - SD_CTRL_GET_RESOLUTION, SD_CTRL_SET_TOP, SD_CTRL_SET_VIDEO_DEF_FPS, SD_CTRL_UPDATE_SPEED, -- cgit v1.2.3