[FFmpeg-devel] [PATCH] avcodec/hevc_ps: Always initialize backup in decode_vui()
Michael Niedermayer
michaelni at gmx.at
Sat Sep 6 21:29:55 CEST 2014
On Sat, Sep 06, 2014 at 01:28:41PM +0200, Christophe Gisquet wrote:
> 2014-09-06 12:40 GMT+02:00 Michael Niedermayer <michaelni at gmx.at>:
> > - // Backup context in case an alternate header is detected
> > - if( get_bits_left(gb) >= 66)
> > - memcpy(&backup, gb, sizeof(backup));
> > + memcpy(&backup, gb, sizeof(backup));
>
> I know people consider obvious comments to waste screen space, but I'd
> prefer more than fewer comments on this.
>
> What about leaving it?
ok
>
> Otherwise, ok, and sorry for the original buggy code.
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
What does censorship reveal? It reveals fear. -- Julian Assange
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140906/7bdd6159/attachment.asc>
More information about the ffmpeg-devel
mailing list