avcodec/movtextdec: add () to CMP() macro to avoid unexpected behavior

Reviewed-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
Michael Niedermayer 2022-02-14 20:20:47 +01:00
parent 36680078ca
commit c182c70658

View File

@ -263,7 +263,7 @@ static int decode_hclr(const uint8_t *tsmb, MovTextContext *m, uint64_t size)
static int styles_equivalent(const StyleBox *a, const StyleBox *b) static int styles_equivalent(const StyleBox *a, const StyleBox *b)
{ {
#define CMP(field) a->field == b->field #define CMP(field) ((a)->field == (b)->field)
return CMP(bold) && CMP(italic) && CMP(underline) && CMP(color) && return CMP(bold) && CMP(italic) && CMP(underline) && CMP(color) &&
CMP(alpha) && CMP(fontsize) && CMP(font_id); CMP(alpha) && CMP(fontsize) && CMP(font_id);
#undef CMP #undef CMP