[FFmpeg-devel] [PATCH] avcodec/dvenc: silence -Wdiscarded-qualifiers

[Clément Bœsch]

On Sat, Sep 19, 2015 at 07:08:01PM -0400, Ganesh Ajjanagadde wrote:
> s->frame is non-const, so this casts them explicitly.
> This suppresses -Wdiscarded-qualifiers seen in e.g
> http://fate.ffmpeg.org/log.cgi?time=20150919100330&log=compile&slot=x86_64-archlinux-gcc-enableshared.
> 
> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
> ---
>  libavcodec/dvenc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/dvenc.c b/libavcodec/dvenc.c
> index 5de12cc..0f4daa1 100644
> --- a/libavcodec/dvenc.c
> +++ b/libavcodec/dvenc.c
> @@ -721,7 +721,7 @@ static int dvvideo_encode_frame(AVCodecContext *c, AVPacket *pkt,
>          return ret;
>  
>      c->pix_fmt                = s->sys->pix_fmt;
> -    s->frame                  = frame;
> +    s->frame                  = (AVFrame *) frame;

Just a note here: I see you tend to like "(type *) var" style. I honestly
don't mind that much, but keep in mind the common way is to stick the var
to the cast (for various reasons such as consistency with most of the
current code base but also because it makes more sense semantically as
well as avoiding  confusion with more complex expressions).

Feel free to commit as is, I won't raise it again, I'm just trying to
prevent an introduction of a more "distributed" code style.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150920/8394b636/attachment.sig>