<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  <title></title>
</head>
<body text="#000000" bgcolor="#ffffff">
After some test there are some issue:<br>
ramdom crash at start up or without sound.<br>
But the real probleme i think is with 5.0 sound.<br>
<br>
In the sample the 2.0 audio out is good <br>
but the 5.0 has low volume and metallic sound<br>
<br>
just try xine <br>
fra piste 5.0<br>
ffmpeg_audio_dec: unknown header with buf type 0x3000000<br>
<br>
qaa 2.0 <br>
is good<br>
<br>
ffmpeg -i 00001.ts <br>
...<br>
Input #0, mpegts, from '00001.ts':<br>
  Duration: 00:02:10.51, start: 10461.634989, bitrate: 6905 kb/s<br>
  Program 132 <br>
    Stream #0.0[0x78]: Video: h264, yuv420p, 1440x1080 [PAR 4:3 DAR
16:9], 50 fps, 50 tbr, 90k tbn, 50 tbc<br>
    Stream #0.1[0x82](fra): Audio: eac3, 48000 Hz, 5.0, s16, 256 kb/s<br>
    Stream #0.2[0x83](qaa): Audio: eac3, 48000 Hz, stereo, s16, 128 kb/s<br>
    Stream #0.3[0x8c](fra): Subtitle: dvbsub<br>
    Stream #0.4[0x8d](fra): Subtitle: dvbsub<br>
<br>
<br>
<br>
<br>
Le 25/03/2010 16:08, Jose Alberto Reguero a écrit :
<blockquote cite="mid:201003251608.20823.jareguero@telefonica.net"
 type="cite">
  <pre wrap="">Patch for xine-lib that don't need to patch remux.c to work.

Jose Alberto

El Jueves, 25 de Marzo de 2010, Jose Alberto Reguero escribió:
  </pre>
  <blockquote type="cite">
    <pre wrap="">Patch against latest xine-lib-1.2. New patch for xineliboutput. Now must
work if you patch remux.c.
You must have the latest xine-lib and xineliboutput. Yesterday both have
changes about eac3.

Jose Alberto

El Jueves, 25 de Marzo de 2010, zaverel escribió:
    </pre>
    <blockquote type="cite">
      <pre wrap="">The typo was on remux.c  =&gt; lost  : and ;

Anyway xine-lib has been updated and your patch don't apply.

Update is for eac3 with mkv

i don't test it yet



corrected remux.c

line 533
case SI::AC3DescriptorTag:
+case SI::EnhancedAC3DescriptorTag:


and in line 191:

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

Le 24/03/2010 22:58, Jose Alberto Reguero a écrit :
      </pre>
      <blockquote type="cite">
        <pre wrap="">I attached a second version of the first patch.
I make the same changes that in the second patch, but maintaining the
logic of the first patch. Also I commented the line:
+//        this-&gt;context-&gt;request_channels = 2;
because your example has 5 channels. If you have trouble with that you
can comment the line again.
Which  typo error has the second patch?

Jose Alberto

El Miércoles, 24 de Marzo de 2010, zaverel escribió:
        </pre>
        <blockquote type="cite">
          <pre wrap="">i 've patched pat.c and now remux.c
and with use xine-lib-1.2 with your second patch (who has typo error)
and that doesn't work.

corrected remux.c

line 533
case SI::AC3DescriptorTag:
+case SI::EnhancedAC3DescriptorTag:


and in line 191:

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



with your previously patch and just pat.c patched with
line 402

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

that worked but not stable.

Is your sample eac3 has |"Spectral extension" ?
because in france dvb-t with eac3 has it
and need a ffmpeg patched for that.
And i test with that sound.

Le 24/03/2010 19:41, Jose Alberto Reguero a écrit :
          </pre>
          <blockquote 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ércoles, 24 de Marzo de 2010, dplu escribió:
            </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 é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>
        </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>
    </blockquote>
  </blockquote>
  <pre wrap="">
  </pre>
  <pre wrap="">
<fieldset class="mimeAttachmentHeader"></fieldset>
_______________________________________________
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>