[FFmpeg-devel] [PATCH 2/6] dv: use smaller type for weight tables

Michael Niedermayer michaelni at gmx.at
Sat Oct 25 17:07:06 CEST 2014


On Sat, Oct 25, 2014 at 01:34:22PM +0200, Reimar Döffinger wrote:
> On Sat, Oct 25, 2014 at 11:19:21AM +0000, Christophe Gisquet wrote:
> > ---
> >  libavcodec/dv.c     |  4 ++--
> >  libavcodec/dvdata.c | 12 ++++++------
> >  libavcodec/dvdata.h | 12 ++++++------
> >  3 files changed, 14 insertions(+), 14 deletions(-)
> 
> Looks good to me.
> I think in all places the uint16_t will be promoted to int
> automatically.
> But it might be good for someone else to double-check that this
> signed->unsigned change doesn't break anything, I consider C
> behaviour a bit too tricky...

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Into a blind darkness they enter who follow after the Ignorance,
they as if into a greater darkness enter who devote themselves
to the Knowledge alone. -- Isha Upanishad
-------------- 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/20141025/118715de/attachment.asc>


More information about the ffmpeg-devel mailing list