[prev in list] [next in list] [prev in thread] [next in thread] 

List:       xine-devel
Subject:    [xine-devel] Broken LATM with xine-lib-1.2
From:       Chris Rankin <rankincj () yahoo ! com>
Date:       2014-12-14 23:10:04
Message-ID: 2073691859.341528.1418598604143.JavaMail.yahoo () jws10095 ! mail ! ne1 ! yahoo ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hi,
I've just tested my DVB-T2 reception with xine, and I'm sorry to report tha=
t the LATM decoding is indeed broken by this commit:
changeset:=C2=A0=C2=A0 12672:63e9924bea7d
user:=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Torsten Jager <t.jager@gmx.=
de>
date:=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 Fri Nov 21 16:35:45 2014 +0=
100
files:=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 src/combined/ffmpeg/ff_audio_dec=
oder.c
description:
ff_audio_decoder: add AAC ADTS probing and parsing.
Some MPEG-TS streams dont align ADTS to TS-PES,
pad ADTS frames to (nearly) fixed bitrate, and/or
wrongly flag all this as LATM.
Somebody please test whether this breaks real LATM.

Reverting this patch does restore the audio.
Cheers,Chris


[Attachment #5 (text/html)]

<html><body><div style="color:#000; background-color:#fff; font-family:HelveticaNeue, \
Helvetica Neue, Helvetica, Arial, Lucida Grande, Sans-Serif;font-size:10px"><div \
dir="ltr" id="yui_3_16_0_1_1418589032333_3590">Hi,</div><div \
id="yui_3_16_0_1_1418589032333_3610" dir="ltr"><br></div><div \
id="yui_3_16_0_1_1418589032333_3609" dir="ltr">I've just tested my DVB-T2 reception \
with xine, and I'm sorry to report that the LATM decoding is indeed broken by this \
commit:</div><div id="yui_3_16_0_1_1418589032333_3704" dir="ltr"><br></div><div \
id="yui_3_16_0_1_1418589032333_3705" dir="ltr">changeset:&nbsp;&nbsp; \
12672:63e9924bea7d<br style="" \
class="">user:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Torsten Jager \
&lt;t.jager@gmx.de&gt;<br style="" \
class="">date:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Fri Nov 21 16:35:45 2014 \
+0100<br style="" class="">files:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; \
src/combined/ffmpeg/ff_audio_decoder.c<br style="" class="">description:<br style="" \
class="">ff_audio_decoder: add AAC ADTS probing and parsing.<br style="" \
class="">Some MPEG-TS streams dont align ADTS to TS-PES,<br style="" class="">pad \
ADTS frames to (nearly) fixed bitrate, and/or<br style="" class="">wrongly flag all \
this as LATM.<br style="" class="">Somebody please test whether this breaks real \
LATM.<br style="" class=""></div><div id="yui_3_16_0_1_1418589032333_3725" \
dir="ltr"><br></div><div id="yui_3_16_0_1_1418589032333_3726" dir="ltr">Reverting \
this patch does restore the audio.</div><div id="yui_3_16_0_1_1418589032333_3873" \
dir="ltr"><br></div><div id="yui_3_16_0_1_1418589032333_3874" \
dir="ltr">Cheers,</div><div id="yui_3_16_0_1_1418589032333_3875" \
dir="ltr">Chris<br></div><div style="" class="" id="yui_3_16_0_1_1418589032333_3705" \
dir="ltr"><br style="" class=""></div></div></body></html>



------------------------------------------------------------------------------
Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server
from Actuate! Instantly Supercharge Your Business Reports and Dashboards
with Interactivity, Sharing, Native Excel Exports, App Integration & more
Get technology previously reserved for billion-dollar corporations, FREE
http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk

_______________________________________________
xine-devel mailing list
xine-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/xine-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic