[MPlayer-dev-eng] [PATCH] codec-cfg.c fixes

Sidik Isani lksi at cfht.hawaii.edu
Mon Nov 11 04:38:19 CET 2002


Hello again -

  This patch goes into the #ifdef CODECS2HTML section of
  codec-cfg.c and makes it possible to build that utility
  again.  It also fixes two cases where *_codecs was set
  to NULL if it was NULL ;-)  Double-check me, please, but
  I think the patch makes it the way it should have been.
  (Not an issue if the function is never called twice.)

Be seeing you,

- Sidik
-------------- next part --------------
diff -ru MPlayer-cvs-pre10.DIST/codec-cfg.c MPlayer-cvs-pre10/codec-cfg.c
--- MPlayer-cvs-pre10.DIST/codec-cfg.c	Wed Oct 16 14:20:44 2002
+++ MPlayer-cvs-pre10/codec-cfg.c	Sun Nov 10 13:42:42 2002
@@ -470,12 +470,12 @@
 	int codec_type;		/* TYPE_VIDEO/TYPE_AUDIO */
 	int tmp, i;
 	
-	// in case we call it secont time
+	// in case we call it a second time
 	if(video_codecs!=NULL)free(video_codecs);
-	else video_codecs=NULL;
+	video_codecs=NULL;
  
  	if(audio_codecs!=NULL)free(audio_codecs);
-	else audio_codecs=NULL;
+	audio_codecs=NULL;
 	
 	nr_vcodecs = 0;
 	nr_acodecs = 0;
@@ -830,6 +830,13 @@
 
 
 #ifdef CODECS2HTML
+/*
+ * Fake out GUI references when building the codecs2html utility.
+ */
+#ifdef HAVE_NEW_GUI
+void gtkMessageBox( int type,char * str ) { return; }
+int use_gui = 0;
+#endif
 
 void wrapline(FILE *f2,char *s){
     int c;
@@ -863,7 +870,7 @@
             case 'd':
                 wrapline(f2,codec->dll); break;
             case 'D':
-                fprintf(f2,"%c",codec->driver==dshow?'+':'-'); break;
+                fprintf(f2,"%c",!strcmp(codec->drv,"dshow")?'+':'-'); break;
             case 'F':
                 for(d=0;d<CODECS_MAX_FOURCC;d++)
                     if(!d || codec->fourcc[d]!=0xFFFFFFFF)
@@ -949,13 +956,13 @@
                     case 0:
                     case 5:
                         if(cl[i].status==CODECS_STATUS_WORKING)
-                            if(!(cl[i].driver==win32 || cl[i].driver==dshow || cl[i].driver==win32ex))
+                            if(!(!strcmp(cl[i].drv,"win32") || !strcmp(cl[i].drv,"dshow") || !strcmp(cl[i].drv,"win32ex")))
                                 parsehtml(f1,f2,&cl[i],section,dshow);
                         break;
                     case 1:
                     case 6:
                         if(cl[i].status==CODECS_STATUS_WORKING)
-                            if(cl[i].driver==win32 || cl[i].driver==dshow || cl[i].driver==win32ex)
+                            if((!strcmp(cl[i].drv,"win32") || !strcmp(cl[i].drv,"dshow") || !strcmp(cl[i].drv,"win32ex")))
                                 parsehtml(f1,f2,&cl[i],section,dshow);
                         break;
                     case 2:


More information about the MPlayer-dev-eng mailing list