[FFmpeg-devel] [PATCH 2/2] Decode DTS XXCH extension

Michael Niedermayer michaelni at gmx.at
Thu Jun 28 22:23:42 CEST 2012


On Tue, Jun 26, 2012 at 08:18:48PM +0100, Nick Brereton wrote:
> On 2012-06-26 12:03, Benjamin Larsson wrote:
> 
> >>@@ -1979,7 +2105,6 @@ static int dca_decode_frame(AVCodecContext
> >>*avctx, void *data,
> >>/* only scan for extensions if ext_descr was unknown or indicated a
> >>* supported XCh extension */
> >>if (s->core_ext_mask < 0 || s->core_ext_mask & (DCA_EXT_XCH |
> >>DCA_EXT_XXCH)) {
> >>-
> >>/* if ext_descr was unknown, clear s->core_ext_mask so that the
> >>* extensions scan can fill it up */
> >>s->core_ext_mask = FFMAX(s->core_ext_mask, 0);
> >
> >This hunk is unneeded.
> 
> Fixed, updated patch attached.
> 
> Nick
> 
> >
> >LGTM otherwise.

applied

thanks

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

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120628/e09b6da9/attachment.asc>


More information about the ffmpeg-devel mailing list