pluginloader: fix compiler warning on win32
authorTim-Philipp Müller <tim.muller@collabora.co.uk>
Wed, 20 Jan 2010 10:58:29 +0000 (10:58 +0000)
committerTim-Philipp Müller <tim.muller@collabora.co.uk>
Wed, 20 Jan 2010 10:58:29 +0000 (10:58 +0000)
Move variable that's only used on unix into the unix block so that
the compiler doesn't complain about the unused variable on win32
(see #597662).

gst/gstpluginloader.c

index 3b87736..fb18f62 100644 (file)
@@ -435,7 +435,6 @@ gboolean
 _gst_plugin_loader_client_run ()
 {
   GstPluginLoader *l;
-  int dup_fd = 0;
 
   l = plugin_loader_new (NULL);
   if (l == NULL)
@@ -445,25 +444,29 @@ _gst_plugin_loader_client_run ()
    * Dup those somewhere better so that plugins printing things
    * won't interfere with anything */
 #ifndef G_OS_WIN32
-  dup_fd = dup (0);             /* STDIN */
-  if (dup_fd == -1) {
-    GST_ERROR ("Failed to start. Could no dup STDIN, errno %d", errno);
-    return FALSE;
-  }
-  l->fd_r.fd = dup_fd;
-  close (0);
+  {
+    int dup_fd;
 
-  dup_fd = dup (1);             /* STDOUT */
-  if (dup_fd == -1) {
-    GST_ERROR ("Failed to start. Could no dup STDOUT, errno %d", errno);
-    return FALSE;
-  }
-  l->fd_w.fd = dup_fd;
-  close (1);
+    dup_fd = dup (0);           /* STDIN */
+    if (dup_fd == -1) {
+      GST_ERROR ("Failed to start. Could no dup STDIN, errno %d", errno);
+      return FALSE;
+    }
+    l->fd_r.fd = dup_fd;
+    close (0);
 
-  /* Dup stderr down to stdout so things that plugins print are visible,
-   * but don't care if it fails */
-  dup2 (2, 1);
+    dup_fd = dup (1);           /* STDOUT */
+    if (dup_fd == -1) {
+      GST_ERROR ("Failed to start. Could no dup STDOUT, errno %d", errno);
+      return FALSE;
+    }
+    l->fd_w.fd = dup_fd;
+    close (1);
+
+    /* Dup stderr down to stdout so things that plugins print are visible,
+     * but don't care if it fails */
+    dup2 (2, 1);
+  }
 #else
   /* FIXME: Use DuplicateHandle and friends on win32 */
   l->fd_w.fd = 1;               /* STDOUT */