[MPlayer-dev-eng] [PATCH] Compile vo_xv without vidmode

daniel carter hedonist at win.co.nz
Thu Mar 7 14:32:26 CET 2002


Just thought i'd better double check my screen size patch to make sure 
--disable-vm will still compile OK, and i think i've run into an 
unrelated bug.

In vo_xv.c (which my patch doesn't touch) at line 325 vm is declared 
wrapped in #ifdef HAVE_XF86VM.
Then at line 344 vm is refered to in a bit of code that is not #ifdefed.
Accordingly i get

vo_xv.c:345: `vm' undeclared (first use in this function)
vo_xv.c:345: (Each undeclared identifier is reported only once
vo_xv.c:345: for each function it appears in.)
make[1]: *** [vo_xv.o] Error 1

I guess 345 needs to be ifdeffed.  Here's my trivial patch to fix it but 
someone outta make sure it's the correct fix.

Index: libvo/vo_xv.c
===================================================================
RCS file: /cvsroot/mplayer/main/libvo/vo_xv.c,v
retrieving revision 1.58
diff -u -r1.58 vo_xv.c
--- libvo/vo_xv.c       23 Feb 2002 01:38:06 -0000      1.58
+++ libvo/vo_xv.c       7 Mar 2002 13:47:47 -0000
@@ -341,7 +341,9 @@
  #endif

   mFullscreen=flags&1;
+#ifdef HAVE_XF86VM
   if( flags&0x02 ) vm = 1;
+#endif
   flip_flag=flags&8;
   num_buffers=vo_doublebuffering?NUM_BUFFERS:1;





More information about the MPlayer-dev-eng mailing list