Don't call ensure_stdout_valid (which would open an unneeded console

* gmessages.c (g_log_default_handler): (Win32:) Don't call
  ensure_stdout_valid (which would open an unneeded console window)
  if we're calling a logging function.  Define ensure_stdout_valid
  as an empty macro on Unix.

* gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
  then try HOMEDRIVE and HOMEPATH.

diff --git a/ChangeLog b/ChangeLog
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/ChangeLog.pre-2-0 b/ChangeLog.pre-2-0
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog.pre-2-0
+++ b/ChangeLog.pre-2-0
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog.pre-2-10
+++ b/ChangeLog.pre-2-10
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog.pre-2-12
+++ b/ChangeLog.pre-2-12
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/ChangeLog.pre-2-2 b/ChangeLog.pre-2-2
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog.pre-2-2
+++ b/ChangeLog.pre-2-2
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/ChangeLog.pre-2-4 b/ChangeLog.pre-2-4
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog.pre-2-4
+++ b/ChangeLog.pre-2-4
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog.pre-2-6
+++ b/ChangeLog.pre-2-6
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8
index 5bb284c..588d419 100644 (file)
--- a/ChangeLog.pre-2-8
+++ b/ChangeLog.pre-2-8
@@ -1,3 +1,13 @@
+Tue Feb 02 00:08:54 1999  Tor Lillqvist  <tml@iki.fi>
+
+       * gmessages.c (g_log_default_handler): (Win32:) Don't call
+       ensure_stdout_valid (which would open an unneeded console window)
+       if we're calling a logging function.  Define ensure_stdout_valid
+       as an empty macro on Unix.
+
+       * gutils.c (g_get_any_init): (Win32:) Trust HOME first if defined,
+       then try HOMEDRIVE and HOMEPATH.
+
 Mon Feb  1 19:04:28 1999  Owen Taylor  <otaylor@redhat.com>
 
        * gmain.c (g_main_iterate): Release the main_loop

diff --git a/glib/gmessages.c b/glib/gmessages.c
index 1ef3344..404e953 100644 (file)
--- a/glib/gmessages.c
+++ b/glib/gmessages.c
@@ -50,12 +50,10 @@ write (FILE       *fd,
 
   return len;
 }
-#endif /* NATIVE_WIN32 */
 
 static void
 ensure_stdout_valid (void)
 {
-#ifdef NATIVE_WIN32
   HANDLE handle;
 
   handle = GetStdHandle (STD_OUTPUT_HANDLE);
@@ -65,8 +63,10 @@ ensure_stdout_valid (void)
       AllocConsole ();
       freopen ("CONOUT$", "w", stdout);
     }
-#endif
 }
+#else
+#define ensure_stdout_valid()  /* Define as empty */
+#endif
        
 
 /* --- structures --- */
@@ -429,7 +429,6 @@ g_log_default_handler (const gchar    *log_domain,
    * DOS prompt.
    */
   fd = stdout;
-  ensure_stdout_valid ();
 #else
   fd = (log_level >= G_LOG_LEVEL_MESSAGE) ? 1 : 2;
 #endif
@@ -450,6 +449,7 @@ g_log_default_handler (const gchar    *log_domain,
          return;
        }
       /* use write(2) for output, in case we are out of memeory */
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, "\n", 1);
@@ -469,6 +469,7 @@ g_log_default_handler (const gchar    *log_domain,
        write (fd, "\n", 1);
       break;
     case G_LOG_LEVEL_CRITICAL:
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, "\n", 1);
@@ -494,6 +495,7 @@ g_log_default_handler (const gchar    *log_domain,
          local_glib_warning_func (message);
          return;
        }
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, "\n", 1);
@@ -519,6 +521,7 @@ g_log_default_handler (const gchar    *log_domain,
          local_glib_message_func (message);
          return;
        }
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));
@@ -535,6 +538,7 @@ g_log_default_handler (const gchar    *log_domain,
        write (fd, "\n", 1);
       break;
     case G_LOG_LEVEL_INFO:
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));
@@ -551,6 +555,7 @@ g_log_default_handler (const gchar    *log_domain,
        write (fd, "\n", 1);
       break;
     case G_LOG_LEVEL_DEBUG:
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));
@@ -570,6 +575,7 @@ g_log_default_handler (const gchar    *log_domain,
       /* we are used for a log level that is not defined by GLib itself,
        * try to make the best out of it.
        */
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));

diff --git a/glib/gutils.c b/glib/gutils.c
index 50e7c36..321a131 100644 (file)
--- a/glib/gutils.c
+++ b/glib/gutils.c
@@ -411,31 +411,33 @@ g_get_any_init (void)
 #endif /* NATIVE_WIN32 */
        }
       
-#ifdef NATIVE_WIN32
-      /* The official way to specify a home directory on NT is
-       * the HOMEDRIVE and HOMEPATH environment variables.
-       *
-       * This is inside #ifdef NATIVE_WIN32 because with the cygwin dll,
-       * HOME should be a POSIX style pathname.
-       */
+      if (!g_home_dir)
+       g_home_dir = g_strdup (g_getenv ("HOME"));
       
-      if (getenv ("HOMEDRIVE") != NULL && getenv ("HOMEPATH") != NULL)
+#ifdef NATIVE_WIN32
+      if (!g_home_dir)
        {
-         gchar *homedrive, *homepath;
-
-         homedrive = g_strdup (g_getenv ("HOMEDRIVE"));
-         homepath = g_strdup (g_getenv ("HOMEPATH"));
-
-         g_home_dir = g_strconcat (homedrive, homepath, NULL);
-         g_free (homedrive);
-         g_free (homepath);
+         /* The official way to specify a home directory on NT is
+          * the HOMEDRIVE and HOMEPATH environment variables.
+          *
+          * This is inside #ifdef NATIVE_WIN32 because with the cygwin dll,
+          * HOME should be a POSIX style pathname.
+          */
+         
+         if (getenv ("HOMEDRIVE") != NULL && getenv ("HOMEPATH") != NULL)
+           {
+             gchar *homedrive, *homepath;
+             
+             homedrive = g_strdup (g_getenv ("HOMEDRIVE"));
+             homepath = g_strdup (g_getenv ("HOMEPATH"));
+             
+             g_home_dir = g_strconcat (homedrive, homepath, NULL);
+             g_free (homedrive);
+             g_free (homepath);
+           }
        }
 #endif /* !NATIVE_WIN32 */
       
-      if (!g_home_dir)
-       g_home_dir = g_strdup (g_getenv ("HOME"));
-      
-      
 #ifdef HAVE_PWD_H
       {
        struct passwd *pw = NULL;

diff --git a/gmessages.c b/gmessages.c
index 1ef3344..404e953 100644 (file)
--- a/gmessages.c
+++ b/gmessages.c
@@ -50,12 +50,10 @@ write (FILE       *fd,
 
   return len;
 }
-#endif /* NATIVE_WIN32 */
 
 static void
 ensure_stdout_valid (void)
 {
-#ifdef NATIVE_WIN32
   HANDLE handle;
 
   handle = GetStdHandle (STD_OUTPUT_HANDLE);
@@ -65,8 +63,10 @@ ensure_stdout_valid (void)
       AllocConsole ();
       freopen ("CONOUT$", "w", stdout);
     }
-#endif
 }
+#else
+#define ensure_stdout_valid()  /* Define as empty */
+#endif
        
 
 /* --- structures --- */
@@ -429,7 +429,6 @@ g_log_default_handler (const gchar    *log_domain,
    * DOS prompt.
    */
   fd = stdout;
-  ensure_stdout_valid ();
 #else
   fd = (log_level >= G_LOG_LEVEL_MESSAGE) ? 1 : 2;
 #endif
@@ -450,6 +449,7 @@ g_log_default_handler (const gchar    *log_domain,
          return;
        }
       /* use write(2) for output, in case we are out of memeory */
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, "\n", 1);
@@ -469,6 +469,7 @@ g_log_default_handler (const gchar    *log_domain,
        write (fd, "\n", 1);
       break;
     case G_LOG_LEVEL_CRITICAL:
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, "\n", 1);
@@ -494,6 +495,7 @@ g_log_default_handler (const gchar    *log_domain,
          local_glib_warning_func (message);
          return;
        }
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, "\n", 1);
@@ -519,6 +521,7 @@ g_log_default_handler (const gchar    *log_domain,
          local_glib_message_func (message);
          return;
        }
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));
@@ -535,6 +538,7 @@ g_log_default_handler (const gchar    *log_domain,
        write (fd, "\n", 1);
       break;
     case G_LOG_LEVEL_INFO:
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));
@@ -551,6 +555,7 @@ g_log_default_handler (const gchar    *log_domain,
        write (fd, "\n", 1);
       break;
     case G_LOG_LEVEL_DEBUG:
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));
@@ -570,6 +575,7 @@ g_log_default_handler (const gchar    *log_domain,
       /* we are used for a log level that is not defined by GLib itself,
        * try to make the best out of it.
        */
+      ensure_stdout_valid ();
       if (log_domain)
        {
          write (fd, log_domain, strlen (log_domain));

diff --git a/gutils.c b/gutils.c
index 50e7c36..321a131 100644 (file)
--- a/gutils.c
+++ b/gutils.c
@@ -411,31 +411,33 @@ g_get_any_init (void)
 #endif /* NATIVE_WIN32 */
        }
       
-#ifdef NATIVE_WIN32
-      /* The official way to specify a home directory on NT is
-       * the HOMEDRIVE and HOMEPATH environment variables.
-       *
-       * This is inside #ifdef NATIVE_WIN32 because with the cygwin dll,
-       * HOME should be a POSIX style pathname.
-       */
+      if (!g_home_dir)
+       g_home_dir = g_strdup (g_getenv ("HOME"));
       
-      if (getenv ("HOMEDRIVE") != NULL && getenv ("HOMEPATH") != NULL)
+#ifdef NATIVE_WIN32
+      if (!g_home_dir)
        {
-         gchar *homedrive, *homepath;
-
-         homedrive = g_strdup (g_getenv ("HOMEDRIVE"));
-         homepath = g_strdup (g_getenv ("HOMEPATH"));
-
-         g_home_dir = g_strconcat (homedrive, homepath, NULL);
-         g_free (homedrive);
-         g_free (homepath);
+         /* The official way to specify a home directory on NT is
+          * the HOMEDRIVE and HOMEPATH environment variables.
+          *
+          * This is inside #ifdef NATIVE_WIN32 because with the cygwin dll,
+          * HOME should be a POSIX style pathname.
+          */
+         
+         if (getenv ("HOMEDRIVE") != NULL && getenv ("HOMEPATH") != NULL)
+           {
+             gchar *homedrive, *homepath;
+             
+             homedrive = g_strdup (g_getenv ("HOMEDRIVE"));
+             homepath = g_strdup (g_getenv ("HOMEPATH"));
+             
+             g_home_dir = g_strconcat (homedrive, homepath, NULL);
+             g_free (homedrive);
+             g_free (homepath);
+           }
        }
 #endif /* !NATIVE_WIN32 */
       
-      if (!g_home_dir)
-       g_home_dir = g_strdup (g_getenv ("HOME"));
-      
-      
 #ifdef HAVE_PWD_H
       {
        struct passwd *pw = NULL;