[MPlayer-dev-eng] [PATCH] header file for stream/stream_vcd.c

Reimar Döffinger Reimar.Doeffinger at gmx.de
Fri Mar 11 19:47:28 CET 2011


On Fri, Mar 11, 2011 at 09:16:36AM +0100, Ingo Brückl wrote:
> Reimar Döffinger wrote on Fri, 11 Mar 2011 08:53:10 +0100:
> 
> > On 11 Mar 2011, at 00:04, Ingo Brückl <ib at wupperonline.de> wrote:
> >> Add missing header file for stream/stream_vcd.c.
> >>
> >> This fixes the compiler warning "no previous prototype for
> >> 'vcd_seek_to_track'" and supplies a declaration for code
> >> using this function.
> 
> > No, you should remove that function and instead use the send_command (or
> > what it is called) infrastructure.
> > The existence of such gui-only functions was the main reason we
> > considered removing the GUI.
> 
> Agreed, but could we please just add it temporarily for the moment being?

No, I don't think there's a sense to that.

> There is a lot rudimentary stuff to do with the GUI code and I can't spend
> time on rewriting, introducing and testing new stuff right now. First I have
> to have an overview on how the GUI works in detail and clean it up elementary
> before I can start seperating it.
> 
> I'll put this topic on my list and will remove all this as soon as possible.

Just use this as a band-aid (untested I admit):
Index: gui/interface.c
===================================================================
--- gui/interface.c     (revision 33029)
+++ gui/interface.c     (working copy)
@@ -67,8 +67,6 @@
 #include "stream/stream_dvd.h"
 #endif
 
-int vcd_seek_to_track(void *vcd, int track);
-
 #include "m_config.h"
 #include "m_option.h"
 
@@ -591,10 +589,8 @@
                    guiIntfStruct.VCDTracks=0;
                    break;
                }
-               for ( i=1;i < 100;i++ )
-                 if ( vcd_seek_to_track( stream->priv,i ) < 0 ) break;
-               vcd_seek_to_track( stream->priv,vcd_track );
-               guiIntfStruct.VCDTracks=--i;
+               stream_control(stream, STREAM_CTRL_GET_NUM_CHAPTERS, &guiIntfStruct.VCDTracks);
+               stream_control(stream, STREAM_CTRL_SEEK_TO_CHAPTER, &vcd_track);
                break;
               }
 #endif


More information about the MPlayer-dev-eng mailing list