Don't call close() on -1

Of course, a proper implementation of close() will just ignore an
invalid parameter silently, and set errno. But apparently the "debug"
version of the Microsoft C library generates some noise in this
case. So avoid that. Thanks to John Emmas for reporting.
This commit is contained in:
Tor Lillqvist 2010-10-07 12:04:52 +03:00
parent 200e3d7307
commit 46390c9d10

View File

@ -886,7 +886,8 @@ test_file_functions (void)
fd = g_mkstemp (template);
if (g_test_verbose() && fd != -1)
g_print ("g_mkstemp works even if template doesn't end in XXXXXX\n");
close (fd);
if (fd != -1)
close (fd);
strcpy (template, "fooXXXXXX");
fd = g_mkstemp (template);
if (fd == -1)