aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTADANO Tokumei <aimingoff@pc.nifty.jp>2023-10-17 22:13:33 +0900
committerJan Ekström <jeebjp@gmail.com>2023-10-29 18:19:58 +0200
commit82faba8a6ce8b6a9a3db635ce0a70495a2b2cf3c (patch)
tree25a68d8cc384b25f21382703eca39e99a0eda3f3
parentff3429991ec1bac1d1b71215402e3d195162e125 (diff)
downloadffmpeg-82faba8a6ce8b6a9a3db635ce0a70495a2b2cf3c.tar.gz
lavc/libaribcaption: switch all `bool` context variables to `int`
On some environments, a `bool` variable is of smaller size than `int`. As AV_OPT_TYPE_BOOL is internally handled as sizeof(int), if a `bool` option was set on such an environment, the memory of following variables would be filled. Additionally, set values may be destroyed by av_opt_copy(). Signed-off-by: TADANO Tokumei <aimingoff@pc.nifty.jp>
-rw-r--r--libavcodec/libaribcaption.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/libavcodec/libaribcaption.c b/libavcodec/libaribcaption.c
index 8a8c8f8cfd..be3328c5c9 100644
--- a/libavcodec/libaribcaption.c
+++ b/libavcodec/libaribcaption.c
@@ -68,14 +68,14 @@ typedef struct ARIBCaptionContext {
int subtitle_type;
int encoding_scheme;
- bool ass_single_rect;
+ int ass_single_rect;
char *font;
- bool replace_fullwidth_ascii;
- bool force_stroke_text;
- bool ignore_background;
- bool ignore_ruby;
+ int replace_fullwidth_ascii;
+ int force_stroke_text;
+ int ignore_background;
+ int ignore_ruby;
float stroke_width;
- bool replace_drcs;
+ int replace_drcs;
int64_t pts;
AVRational time_base;