[mythtv] Re: [mythtv-commits] mythtv siparser changes....

[Stuart Auchterlonie]

On Tue, May 03, 2005 at 05:45:12PM +0100, Neale Swinnerton wrote:
> 
> 
> Stuart Auchterlonie wrote:
> <snip>
> > That's the freeview /xtraview patch that was sent to the list a couple
> > of weeks ago. There is however a better way to do it than the original
> > patch ;-)
> 
> What's the better way?

Okay, the old version was messy because it assumed that the
private data streams were either video or audio. I identified an
id byte in the private streams which relate to video and audio.

The same byte can also be used to identify subtitle descriptors.
Someone else is already looking into dvb subtitles.


Oh and there's a patch as well.....
Stuart

-------------- next part --------------
Index: libs/libmythtv/siparser.cpp
===================================================================
--- libs/libmythtv/siparser.cpp	(revision 19)
+++ libs/libmythtv/siparser.cpp	(working copy)
@@ -724,6 +724,21 @@
             case 0x04:
                 e.Type = ES_TYPE_AUDIO_MPEG2;
                 break;
+            case 0x06:
+                switch (buffer[pos+5])
+                {
+                    case 0x0A: // Audio in PES Private Section
+                        e.Type = ES_TYPE_AUDIO_MPEG1;
+                        break;
+                    case 0x52: // Video in PES Private Section
+                        e.Type = ES_TYPE_VIDEO_MPEG1;
+                        break;
+                    default:
+                        e.Type = ES_TYPE_DATA;
+                        break;
+                }
+                p.PCRPID = e.PID;
+                break;
             case 0x08:
             case 0x0B:
                 e.Type = ES_TYPE_DATA;