<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=ISO-8859-1"
 http-equiv="Content-Type">
  <title></title>
</head>
<body text="#000000" bgcolor="#ffffff">
i 've patched pat.c and now remux.c<br>
and with use xine-lib-1.2 with your second patch (who has typo error)
and that doesn't work.<br>
<br>
<small>corrected</small> <small>remux.c</small><br>
<pre wrap="">line 533
case SI::AC3DescriptorTag:
+case SI::EnhancedAC3DescriptorTag:


and in line 191:

-Target[i++] = SI::AC3DescriptorTag;
+Target[i++] = SI::EnhancedAC3DescriptorTag;</pre>
<br>
<br>
with your previously patch and just pat.c patched with <br>
<small>line 402</small><br>
<pre wrap="">case SI::AC3DescriptorTag:
+case SI:EnhancedAC3DescriptorTag:

<big>that worked but not stable.</big>

<big>Is your sample eac3 has </big><code><big>"Spectral extension" ?
because in france dvb-t with eac3 has it
and need a ffmpeg patched for that.
And i test with that sound.
</big></code></pre>
<br>
<br>
Le 24/03/2010 19:41, Jose Alberto Reguero a &eacute;crit&nbsp;:
<blockquote cite="mid:201003241941.43539.jareguero@telefonica.net"
 type="cite">
  <pre wrap="">It works here with a old sample of tdt with eac3. Have you patch also remux.c?
You need to change in line 533:

case SI::AC3DescriptorTag:
+case SI:EnhancedAC3DescriptorTag:

and in line 191:

-Target[i++] = SI::AC3DescriptorTag;
+Target[i++] = SI::EnhancedAC3DescriptorTag

Jose Alberto

El Mi&eacute;rcoles, 24 de Marzo de 2010, dplu escribi&oacute;:
  </pre>
  <blockquote type="cite">
    <pre wrap="">Hi

I have made previous test with the version release by Petri Hintukainen

And I notice this part is not working like it should

-    if((m-&gt;descriptor_tag == STREAM_AUDIO_AC3) ||    /* ac3 - raw */
+    if(m-&gt;descriptor_tag == HDMV_AUDIO_84_EAC3) {
+      m-&gt;content   = p;
+      m-&gt;size = packet_len;
+      m-&gt;type |= BUF_AUDIO_EAC3;
+      return 1;
+
+    } else if((m-&gt;descriptor_tag == STREAM_AUDIO_AC3) ||    /* ac3 - raw
*/

unfortunaletly, in AC3 or E-AC3 , the descriptor tag is STREAM_AUDIO_AC3,
so the program never run the first if (installed a debug printf here)

It seems that your first approach (at least what I understood) by forcing
the decoding of all AC-3 stream by ffmepg instead of internal lib was nice
but generate violent segfault on libavcodec

Hope this help you

Best regards

PS : Sorry to pollute the vdr mailing list (not subscribed to ffmpeg or
xine-lib....)

Le Wednesday 24 March 2010 18:41:14 zaverel, vous avez &eacute;crit :
    </pre>
    <blockquote type="cite">
      <pre wrap="">hello

 your second patch doesn't work  : no sound on eac3 channel

if you want there is a sample of vdr hd-eac3 recording here:

<a class="moz-txt-link-freetext" href="http://dl.free.fr/ew4rJddM8">http://dl.free.fr/ew4rJddM8</a>

 103mo

else , i don't know what mailing-list is the more indicate for debuging

the vdr or xine-dev mailing-list ?

_______________________________________________
vdr mailing list
<a class="moz-txt-link-abbreviated" href="mailto:vdr@linuxtv.org">vdr@linuxtv.org</a>
<a class="moz-txt-link-freetext" href="http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr">http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr</a>
      </pre>
    </blockquote>
    <pre wrap="">
_______________________________________________
vdr mailing list
<a class="moz-txt-link-abbreviated" href="mailto:vdr@linuxtv.org">vdr@linuxtv.org</a>
<a class="moz-txt-link-freetext" href="http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr">http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr</a>
    </pre>
  </blockquote>
  <pre wrap="">

_______________________________________________
vdr mailing list
<a class="moz-txt-link-abbreviated" href="mailto:vdr@linuxtv.org">vdr@linuxtv.org</a>
<a class="moz-txt-link-freetext" href="http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr">http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr</a>


  </pre>
</blockquote>
<br>
</body>
</html>