pluginloader: fix compiler warning on win32

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).
This commit is contained in:
Tim-Philipp Müller 2010-01-20 10:58:29 +00:00
parent 8967604053
commit f3d579362c

View 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 */