[FFmpeg-cvslog] avfilter/blend_modes: Always preserve constness
Andreas Rheinhardt
git at videolan.org
Thu Sep 7 21:57:51 EEST 2023
ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Thu Sep 7 15:30:30 2023 +0200| [44dcc4d6060c6dfffb60777396ee8ed8bd1fed76] | committer: Andreas Rheinhardt
avfilter/blend_modes: Always preserve constness
These casts cast const away temporarily; they are safe, because
the pointers that are initialized point to const data. But this
is nevertheless not nice and leads to warnings when using
-Wcast-qual. blend_modes.c generates 546 (2*39*7) such warnings
which is the majority of such warnings for FFmpeg as a whole.
vf_blend.c and vf_blend_init.h also use this pattern;
they have also been changed.
Reviewed-by: Paul B Mahol <onemda at gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=44dcc4d6060c6dfffb60777396ee8ed8bd1fed76
---
libavfilter/blend_modes.c | 4 ++--
libavfilter/vf_blend.c | 4 ++--
libavfilter/vf_blend_init.h | 4 ++--
3 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/libavfilter/blend_modes.c b/libavfilter/blend_modes.c
index e2be676243..65c5e6f890 100644
--- a/libavfilter/blend_modes.c
+++ b/libavfilter/blend_modes.c
@@ -93,8 +93,8 @@ static void fn0(NAME)(const uint8_t *_top, ptrdiff_t top_linesize, \
ptrdiff_t width, ptrdiff_t height, \
FilterParams *param, double *values, int starty) \
{ \
- const PIXEL *top = (PIXEL *)_top; \
- const PIXEL *bottom = (PIXEL *)_bottom; \
+ const PIXEL *top = (const PIXEL *)_top; \
+ const PIXEL *bottom = (const PIXEL *)_bottom; \
PIXEL *dst = (PIXEL *)_dst; \
const float opacity = param->opacity; \
\
diff --git a/libavfilter/vf_blend.c b/libavfilter/vf_blend.c
index dfe2b8b174..7100d9f372 100644
--- a/libavfilter/vf_blend.c
+++ b/libavfilter/vf_blend.c
@@ -133,8 +133,8 @@ static void blend_expr_## name(const uint8_t *_top, ptrdiff_t top_linesize,
ptrdiff_t width, ptrdiff_t height, \
FilterParams *param, double *values, int starty) \
{ \
- const type *top = (type*)_top; \
- const type *bottom = (type*)_bottom; \
+ const type *top = (const type*)_top; \
+ const type *bottom = (const type*)_bottom; \
type *dst = (type*)_dst; \
AVExpr *e = param->e; \
int y, x; \
diff --git a/libavfilter/vf_blend_init.h b/libavfilter/vf_blend_init.h
index f531338a54..d24f178032 100644
--- a/libavfilter/vf_blend_init.h
+++ b/libavfilter/vf_blend_init.h
@@ -82,8 +82,8 @@ static void blend_normal_##name(const uint8_t *_top, ptrdiff_t top_linesize,
ptrdiff_t width, ptrdiff_t height, \
FilterParams *param, double *values, int starty) \
{ \
- const type *top = (type*)_top; \
- const type *bottom = (type*)_bottom; \
+ const type *top = (const type*)_top; \
+ const type *bottom = (const type*)_bottom; \
type *dst = (type*)_dst; \
const float opacity = param->opacity; \
\
More information about the ffmpeg-cvslog
mailing list