fix some warnings (define order)

remove the static 'i' variable for the module


git-svn-id: svn://svn.mplayerhq.hu/mplayer/trunk@10856 b3059339-0415-0410-9bf9-f77b7e298cf2

1 files changed, 21 insertions, 13 deletions

diff --git a/libvo/vo_xv.c b/libvo/vo_xv.c
index 56f023d420..94a96bfeb2 100644
--- a/libvo/vo_xv.c
+++ b/libvo/vo_xv.c
@@ -50,12 +50,26 @@ static vo_info_t info =
 
 LIBVO_EXTERN(xv)
 
+#ifdef HAVE_SHM
+#include <sys/ipc.h>
+#include <sys/shm.h>
+#include <X11/extensions/XShm.h>
+
+/* since it doesn't seem to be defined on some platforms */
+int XShmGetEventBase(Display*);
+
+static XShmSegmentInfo Shminfo[NUM_BUFFERS];
+static int Shmem_Flag;
+#endif
+
+// Note: depends on the inclusion of X11/extensions/XShm.h
 #include <X11/extensions/Xv.h>
 #include <X11/extensions/Xvlib.h>
+
 // FIXME: dynamically allocate this stuff
 static void allocate_xvimage(int);
 static unsigned int ver,rel,req,ev,err;
-static unsigned int formats, adaptors,i,xv_port,xv_format;
+static unsigned int formats, adaptors, xv_port, xv_format;
 static XvAdaptorInfo        *ai = NULL;
 static XvImageFormatValues  *fo;
 
@@ -64,17 +78,6 @@ static int current_ip_buf=0;
 static int num_buffers=1; // default
 static XvImage* xvimage[NUM_BUFFERS];
 
-#ifdef HAVE_SHM
-#include <sys/ipc.h>
-#include <sys/shm.h>
-#include <X11/extensions/XShm.h>
-
-/* since it doesn't seem to be defined on some platforms */
-int XShmGetEventBase(Display*);
-
-static XShmSegmentInfo Shminfo[NUM_BUFFERS];
-static int Shmem_Flag;
-#endif
 
 static uint32_t image_width;
 static uint32_t image_height;
@@ -163,13 +166,16 @@ static uint32_t config(uint32_t width, uint32_t height, uint32_t d_width, uint32
  num_buffers=vo_doublebuffering?(vo_directrendering?NUM_BUFFERS:2):1;
 
    /* check image formats */
+   {
+     unsigned int i;
+
      xv_format=0;
      for(i = 0; i < formats; i++){
        mp_msg(MSGT_VO,MSGL_V,"Xvideo image format: 0x%x (%4.4s) %s\n", fo[i].id,(char*)&fo[i].id, (fo[i].format == XvPacked) ? "packed" : "planar");
        if (fo[i].id == format) xv_format = fo[i].id;
      }
      if (!xv_format) return -1;
- 
+   }
  aspect_save_screenres(vo_screenwidth,vo_screenheight);
 
 #ifdef HAVE_NEW_GUI
@@ -565,6 +571,7 @@ static uint32_t get_image(mp_image_t *mpi){
 
 static uint32_t query_format(uint32_t format)
 {
+    uint32_t i;
     int flag=VFCAP_CSP_SUPPORTED|VFCAP_CSP_SUPPORTED_BY_HW|
 	    VFCAP_HWSCALE_UP|VFCAP_HWSCALE_DOWN|VFCAP_OSD|VFCAP_ACCEPT_STRIDE; // FIXME! check for DOWN
    /* check image formats */
@@ -592,6 +599,7 @@ static uint32_t preinit(const char *arg)
 {
     XvPortID xv_p;
     int busy_ports=0;
+    unsigned int i;
     
     xv_port = 0;