[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