#include <gerror.h>
authorHavoc Pennington <hp@redhat.com>
Wed, 12 Jul 2000 21:06:16 +0000 (21:06 +0000)
committerHavoc Pennington <hp@src.gnome.org>
Wed, 12 Jul 2000 21:06:16 +0000 (21:06 +0000)
2000-07-12  Havoc Pennington  <hp@redhat.com>

* glib.h: #include <gerror.h>

* Makefile.am (include_HEADERS): Add gerror.h
(libglib_la_SOURCES): Add gbacktrace.c

* gbacktrace.c: Move g_on_error_query() in here (moved on the
server, so history is preserved)

* gerror.h: GError interface

* gerror.c: GError implementation replaces stuff that's now in
gbacktrace.c

16 files changed:
ChangeLog
ChangeLog.pre-2-0
ChangeLog.pre-2-10
ChangeLog.pre-2-12
ChangeLog.pre-2-2
ChangeLog.pre-2-4
ChangeLog.pre-2-6
ChangeLog.pre-2-8
Makefile.am
gerror.c
gerror.h [new file with mode: 0644]
glib.h
glib/Makefile.am
glib/gerror.c
glib/gerror.h [new file with mode: 0644]
glib/glib.h

index a75eb2d..62077e1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index a75eb2d..62077e1 100644 (file)
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index a75eb2d..62077e1 100644 (file)
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index a75eb2d..62077e1 100644 (file)
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index a75eb2d..62077e1 100644 (file)
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index a75eb2d..62077e1 100644 (file)
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index a75eb2d..62077e1 100644 (file)
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index a75eb2d..62077e1 100644 (file)
@@ -1,3 +1,18 @@
+2000-07-12  Havoc Pennington  <hp@redhat.com>
+
+       * glib.h: #include <gerror.h>
+
+       * Makefile.am (include_HEADERS): Add gerror.h
+       (libglib_la_SOURCES): Add gbacktrace.c
+
+       * gbacktrace.c: Move g_on_error_query() in here (moved on the 
+       server, so history is preserved)
+
+       * gerror.h: GError interface
+
+       * gerror.c: GError implementation replaces stuff that's now in 
+       gbacktrace.c
+
 Sun Jul  9 21:20:45 2000  Owen Taylor  <otaylor@redhat.com>
 
        * gunicode.h: Include stddef.h instead of stdlib.h
index f47fff2..ae5fedc 100644 (file)
@@ -38,6 +38,7 @@ lib_LTLIBRARIES = libglib.la
 libglib_la_SOURCES =           \
        garray.c                \
        gasyncqueue.c           \
+       gbacktrace.c            \
        gcache.c                \
        gcompletion.c           \
        gdataset.c              \
@@ -71,7 +72,7 @@ libglib_la_SOURCES =          \
        gunidecomp.c            \
        gutils.c
 
-include_HEADERS = glib.h glib-object.h gunicode.h
+include_HEADERS = glib.h glib-object.h gunicode.h gerror.h
 
 configexecincludedir = $(pkglibdir)/include
 #configexecinclude_DATA = glibconfig.h
index 7fa7768..42ea1b6 100644 (file)
--- a/gerror.c
+++ b/gerror.c
  * GLib at ftp://ftp.gtk.org/pub/gtk/. 
  */
 
-/* 
- * MT safe ; except for g_on_error_stack_trace, but who wants thread safety 
- * then
- */
-
-#ifdef HAVE_CONFIG_H
-#include <config.h>
-#endif
-
-#include <signal.h>
-#include <stdarg.h>
-#include <stdio.h>
-#include <stdlib.h>
 #include "glib.h"
-#ifdef HAVE_SYS_TIME_H
-#include <sys/time.h>
-#endif
-#ifdef HAVE_SYS_TIMES_H
-#include <sys/times.h>
-#endif
-#include <sys/types.h>
 
-#include <time.h>
-#ifdef HAVE_UNISTD_H
-#include <unistd.h>
-#endif
+static GError* 
+g_error_new_valist(GQuark         domain,
+                   gint           code,
+                   const gchar   *format,
+                   va_list        args)
+{
+  GError *error;
+  
+  error = g_new (GError, 1);
+  
+  error->domain = domain;
+  error->code = code;
+  error->message = g_strdup_vprintf (format, args);
+  
+  return error;
+}
+
+GError*
+g_error_new (GQuark       domain,
+             gint         code,
+             const gchar *format,
+             ...)
+{
+  GError* error;
+  va_list args;
 
-#ifdef HAVE_SYS_SELECT_H
-#include <sys/select.h>
-#endif /* HAVE_SYS_SELECT_H */
+  g_return_val_if_fail (format != NULL, NULL);
+  g_return_val_if_fail (domain != 0, NULL);
 
-#ifdef STDC_HEADERS
-#include <string.h> /* for bzero on BSD systems */
-#endif
+  va_start (args, format);
+  error = g_error_new_valist (domain, code, format, args);
+  va_end (args);
 
-#ifdef G_OS_WIN32
-#  define STRICT               /* Strict typing, please */
-#  include <windows.h>
-#  include <process.h>         /* For _getpid() */
-#endif
+  return error;
+}
 
-#ifndef NO_FD_SET
-#  define SELECT_MASK fd_set
-#else
-#  if defined(_IBMR2)
-#    define SELECT_MASK void
-#  else
-#    define SELECT_MASK int
-#  endif
-#endif
+GError*
+g_error_new_literal (GQuark         domain,
+                     gint           code,
+                     const gchar   *message)
+{
+  GError* err;
 
+  g_return_val_if_fail (message != NULL, NULL);
+  g_return_val_if_fail (domain != 0, NULL);
 
-static void stack_trace (char **args);
+  err = g_new (GError, 1);
 
-extern volatile gboolean glib_on_error_halt;
-volatile gboolean glib_on_error_halt = TRUE;
+  err->domain = domain;
+  err->code = code;
+  err->message = g_strdup (message);
+  
+  return err;
+}
 
 void
-g_on_error_query (const gchar *prg_name)
+g_error_free (GError *error)
 {
-#ifndef G_OS_WIN32
-  static const gchar *query1 = "[E]xit, [H]alt";
-  static const gchar *query2 = ", show [S]tack trace";
-  static const gchar *query3 = " or [P]roceed";
-  gchar buf[16];
+  g_return_if_fail (error != NULL);  
 
-  if (!prg_name)
-    prg_name = g_get_prgname ();
-  
- retry:
-  
-  if (prg_name)
-    fprintf (stdout,
-            "%s (pid:%u): %s%s%s: ",
-            prg_name,
-            (guint) getpid (),
-            query1,
-            query2,
-            query3);
-  else
-    fprintf (stdout,
-            "(process:%u): %s%s: ",
-            (guint) getpid (),
-            query1,
-            query3);
-  fflush (stdout);
-  
-  if (isatty(0) && isatty(1))
-    fgets (buf, 8, stdin); 
-  else
-    strcpy (buf, "E\n");
+  g_free (error->message);
 
-  if ((buf[0] == 'E' || buf[0] == 'e')
-      && buf[1] == '\n')
-    _exit (0);
-  else if ((buf[0] == 'P' || buf[0] == 'p')
-          && buf[1] == '\n')
-    return;
-  else if (prg_name
-          && (buf[0] == 'S' || buf[0] == 's')
-          && buf[1] == '\n')
-    {
-      g_on_error_stack_trace (prg_name);
-      goto retry;
-    }
-  else if ((buf[0] == 'H' || buf[0] == 'h')
-          && buf[1] == '\n')
-    {
-      while (glib_on_error_halt)
-       ;
-      glib_on_error_halt = TRUE;
-      return;
-    }
-  else
-    goto retry;
-#else
-  if (!prg_name)
-    prg_name = g_get_prgname ();
-  
-  MessageBox (NULL, "g_on_error_query called, program terminating",
-             (prg_name && *prg_name) ? prg_name : NULL,
-             MB_OK|MB_ICONERROR);
-  _exit(0);
-#endif
+  g_free (error);
 }
 
-void
-g_on_error_stack_trace (const gchar *prg_name)
+GError*
+g_error_copy (const GError *error)
 {
-#ifdef G_OS_UNIX
-  pid_t pid;
-  gchar buf[16];
-  gchar *args[4] = { "gdb", NULL, NULL, NULL };
+  GError *copy;
+  
+  g_return_val_if_fail (error != NULL, NULL);
 
-  if (!prg_name)
-    return;
+  copy = g_new (GError, 1);
 
-  sprintf (buf, "%u", (guint) getpid ());
+  *copy = *error;
 
-  args[1] = (gchar*) prg_name;
-  args[2] = buf;
+  copy->message = g_strdup (error->message);
 
-  pid = fork ();
-  if (pid == 0)
-    {
-      stack_trace (args);
-      _exit (0);
-    }
-  else if (pid == (pid_t) -1)
-    {
-      perror ("unable to fork gdb");
-      return;
-    }
-  
-  while (glib_on_error_halt)
-    ;
-  glib_on_error_halt = TRUE;
-#else
-  abort ();
-#endif
+  return copy;
 }
 
-static gboolean stack_trace_done = FALSE;
-
-static void
-stack_trace_sigchld (int signum)
+gboolean
+g_error_matches (const GError *error,
+                 GQuark        domain,
+                 gint          code)
 {
-  stack_trace_done = TRUE;
+  return error &&
+    error->domain == domain &&
+    error->code == code;
 }
 
-static void
-stack_trace (char **args)
+void
+g_set_error (GError     **err,
+             GQuark       domain,
+             gint         code,
+             const gchar *format,
+             ...)
 {
-#ifdef G_OS_UNIX
-  pid_t pid;
-  int in_fd[2];
-  int out_fd[2];
-  SELECT_MASK fdset;
-  SELECT_MASK readset;
-  struct timeval tv;
-  int sel, index, state;
-  char buffer[256];
-  char c;
-
-  stack_trace_done = FALSE;
-  signal (SIGCHLD, stack_trace_sigchld);
-
-  if ((pipe (in_fd) == -1) || (pipe (out_fd) == -1))
-    {
-      perror ("unable to open pipe");
-      _exit (0);
-    }
-
-  pid = fork ();
-  if (pid == 0)
-    {
-      close (0); dup (in_fd[0]);   /* set the stdin to the in pipe */
-      close (1); dup (out_fd[1]);  /* set the stdout to the out pipe */
-      close (2); dup (out_fd[1]);  /* set the stderr to the out pipe */
+  va_list args;
 
-      execvp (args[0], args);      /* exec gdb */
-      perror ("exec failed");
-      _exit (0);
-    }
-  else if (pid == (pid_t) -1)
-    {
-      perror ("unable to fork");
-      _exit (0);
-    }
-
-  FD_ZERO (&fdset);
-  FD_SET (out_fd[0], &fdset);
-
-  write (in_fd[1], "backtrace\n", 10);
-  write (in_fd[1], "p x = 0\n", 8);
-  write (in_fd[1], "quit\n", 5);
+  if (err == NULL)
+    return;
 
-  index = 0;
-  state = 0;
+  if (*err != NULL)
+    g_warning ("GError set over the top of a previous GError or uninitialized memory.\n"
+               "This indicates a bug in someone's code. You must ensure an error is NULL before it's set.");
+  
+  va_start (args, format);
+  *err = g_error_new_valist (domain, code, format, args);
+  va_end (args);
+}
 
-  while (1)
+void
+g_clear_error (GError **err)
+{
+  if (err && *err)
     {
-      readset = fdset;
-      tv.tv_sec = 1;
-      tv.tv_usec = 0;
-
-      sel = select (FD_SETSIZE, &readset, NULL, NULL, &tv);
-      if (sel == -1)
-        break;
-
-      if ((sel > 0) && (FD_ISSET (out_fd[0], &readset)))
-        {
-          if (read (out_fd[0], &c, 1))
-            {
-              switch (state)
-                {
-                case 0:
-                  if (c == '#')
-                    {
-                      state = 1;
-                      index = 0;
-                      buffer[index++] = c;
-                    }
-                  break;
-                case 1:
-                  buffer[index++] = c;
-                  if ((c == '\n') || (c == '\r'))
-                    {
-                      buffer[index] = 0;
-                      fprintf (stdout, "%s", buffer);
-                      state = 0;
-                      index = 0;
-                    }
-                  break;
-                default:
-                  break;
-                }
-            }
-        }
-      else if (stack_trace_done)
-        break;
+      g_error_free (*err);
+      *err = NULL;
     }
-
-  close (in_fd[0]);
-  close (in_fd[1]);
-  close (out_fd[0]);
-  close (out_fd[1]);
-  _exit (0);
-#else
-  abort ();
-#endif
 }
diff --git a/gerror.h b/gerror.h
new file mode 100644 (file)
index 0000000..a3a07d4
--- /dev/null
+++ b/gerror.h
@@ -0,0 +1,72 @@
+/* gerror.h - Error reporting system
+ *
+ *  Copyright 2000 Red Hat, Inc.
+ *
+ * The Gnome Library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * The Gnome Library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License along with the Gnome Library; see the file COPYING.LIB.  If not,
+ * write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ *   Boston, MA 02111-1307, USA.
+ */
+
+#ifndef __GERROR_H__
+#define __GERROR_H__
+
+#ifdef __cplusplus
+extern "C"
+{
+#endif
+
+typedef struct _GError GError;
+
+struct _GError
+{
+  GQuark       domain;
+  gint         code;
+  gchar       *message;
+};
+
+GError*  g_error_new           (GQuark         domain,
+                                gint           code,
+                                const gchar   *format,
+                                ...) G_GNUC_PRINTF (3, 4);
+
+GError*  g_error_new_literal   (GQuark         domain,
+                                gint           code,
+                                const gchar   *message);
+
+void     g_error_free          (GError        *error);
+GError*  g_error_copy          (const GError  *error);
+
+gboolean g_error_matches       (const GError  *error,
+                                GQuark         domain,
+                                gint           code);
+
+/* if (err) *err = g_error_new(domain, code, format, ...), also has
+ * some sanity checks.
+ */
+void     g_set_error           (GError       **err,
+                                GQuark         domain,
+                                gint           code,
+                                const gchar   *format,
+                                ...) G_GNUC_PRINTF (4, 5);
+
+/* if (err && *err) { g_error_free(*err); *err = NULL; } */
+void     g_clear_error         (GError       **err);
+
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* __GERROR_H__ */
+
diff --git a/glib.h b/glib.h
index d7ac6d5..31f1e1a 100644 (file)
--- a/glib.h
+++ b/glib.h
@@ -3326,6 +3326,7 @@ guint           g_thread_pool_get_num_unused_threads (void);
 void            g_thread_pool_stop_unused_threads    (void);
 
 #include <gunicode.h>
+#include <gerror.h>
 
 #ifdef __cplusplus
 }
index f47fff2..ae5fedc 100644 (file)
@@ -38,6 +38,7 @@ lib_LTLIBRARIES = libglib.la
 libglib_la_SOURCES =           \
        garray.c                \
        gasyncqueue.c           \
+       gbacktrace.c            \
        gcache.c                \
        gcompletion.c           \
        gdataset.c              \
@@ -71,7 +72,7 @@ libglib_la_SOURCES =          \
        gunidecomp.c            \
        gutils.c
 
-include_HEADERS = glib.h glib-object.h gunicode.h
+include_HEADERS = glib.h glib-object.h gunicode.h gerror.h
 
 configexecincludedir = $(pkglibdir)/include
 #configexecinclude_DATA = glibconfig.h
index 7fa7768..42ea1b6 100644 (file)
  * GLib at ftp://ftp.gtk.org/pub/gtk/. 
  */
 
-/* 
- * MT safe ; except for g_on_error_stack_trace, but who wants thread safety 
- * then
- */
-
-#ifdef HAVE_CONFIG_H
-#include <config.h>
-#endif
-
-#include <signal.h>
-#include <stdarg.h>
-#include <stdio.h>
-#include <stdlib.h>
 #include "glib.h"
-#ifdef HAVE_SYS_TIME_H
-#include <sys/time.h>
-#endif
-#ifdef HAVE_SYS_TIMES_H
-#include <sys/times.h>
-#endif
-#include <sys/types.h>
 
-#include <time.h>
-#ifdef HAVE_UNISTD_H
-#include <unistd.h>
-#endif
+static GError* 
+g_error_new_valist(GQuark         domain,
+                   gint           code,
+                   const gchar   *format,
+                   va_list        args)
+{
+  GError *error;
+  
+  error = g_new (GError, 1);
+  
+  error->domain = domain;
+  error->code = code;
+  error->message = g_strdup_vprintf (format, args);
+  
+  return error;
+}
+
+GError*
+g_error_new (GQuark       domain,
+             gint         code,
+             const gchar *format,
+             ...)
+{
+  GError* error;
+  va_list args;
 
-#ifdef HAVE_SYS_SELECT_H
-#include <sys/select.h>
-#endif /* HAVE_SYS_SELECT_H */
+  g_return_val_if_fail (format != NULL, NULL);
+  g_return_val_if_fail (domain != 0, NULL);
 
-#ifdef STDC_HEADERS
-#include <string.h> /* for bzero on BSD systems */
-#endif
+  va_start (args, format);
+  error = g_error_new_valist (domain, code, format, args);
+  va_end (args);
 
-#ifdef G_OS_WIN32
-#  define STRICT               /* Strict typing, please */
-#  include <windows.h>
-#  include <process.h>         /* For _getpid() */
-#endif
+  return error;
+}
 
-#ifndef NO_FD_SET
-#  define SELECT_MASK fd_set
-#else
-#  if defined(_IBMR2)
-#    define SELECT_MASK void
-#  else
-#    define SELECT_MASK int
-#  endif
-#endif
+GError*
+g_error_new_literal (GQuark         domain,
+                     gint           code,
+                     const gchar   *message)
+{
+  GError* err;
 
+  g_return_val_if_fail (message != NULL, NULL);
+  g_return_val_if_fail (domain != 0, NULL);
 
-static void stack_trace (char **args);
+  err = g_new (GError, 1);
 
-extern volatile gboolean glib_on_error_halt;
-volatile gboolean glib_on_error_halt = TRUE;
+  err->domain = domain;
+  err->code = code;
+  err->message = g_strdup (message);
+  
+  return err;
+}
 
 void
-g_on_error_query (const gchar *prg_name)
+g_error_free (GError *error)
 {
-#ifndef G_OS_WIN32
-  static const gchar *query1 = "[E]xit, [H]alt";
-  static const gchar *query2 = ", show [S]tack trace";
-  static const gchar *query3 = " or [P]roceed";
-  gchar buf[16];
+  g_return_if_fail (error != NULL);  
 
-  if (!prg_name)
-    prg_name = g_get_prgname ();
-  
- retry:
-  
-  if (prg_name)
-    fprintf (stdout,
-            "%s (pid:%u): %s%s%s: ",
-            prg_name,
-            (guint) getpid (),
-            query1,
-            query2,
-            query3);
-  else
-    fprintf (stdout,
-            "(process:%u): %s%s: ",
-            (guint) getpid (),
-            query1,
-            query3);
-  fflush (stdout);
-  
-  if (isatty(0) && isatty(1))
-    fgets (buf, 8, stdin); 
-  else
-    strcpy (buf, "E\n");
+  g_free (error->message);
 
-  if ((buf[0] == 'E' || buf[0] == 'e')
-      && buf[1] == '\n')
-    _exit (0);
-  else if ((buf[0] == 'P' || buf[0] == 'p')
-          && buf[1] == '\n')
-    return;
-  else if (prg_name
-          && (buf[0] == 'S' || buf[0] == 's')
-          && buf[1] == '\n')
-    {
-      g_on_error_stack_trace (prg_name);
-      goto retry;
-    }
-  else if ((buf[0] == 'H' || buf[0] == 'h')
-          && buf[1] == '\n')
-    {
-      while (glib_on_error_halt)
-       ;
-      glib_on_error_halt = TRUE;
-      return;
-    }
-  else
-    goto retry;
-#else
-  if (!prg_name)
-    prg_name = g_get_prgname ();
-  
-  MessageBox (NULL, "g_on_error_query called, program terminating",
-             (prg_name && *prg_name) ? prg_name : NULL,
-             MB_OK|MB_ICONERROR);
-  _exit(0);
-#endif
+  g_free (error);
 }
 
-void
-g_on_error_stack_trace (const gchar *prg_name)
+GError*
+g_error_copy (const GError *error)
 {
-#ifdef G_OS_UNIX
-  pid_t pid;
-  gchar buf[16];
-  gchar *args[4] = { "gdb", NULL, NULL, NULL };
+  GError *copy;
+  
+  g_return_val_if_fail (error != NULL, NULL);
 
-  if (!prg_name)
-    return;
+  copy = g_new (GError, 1);
 
-  sprintf (buf, "%u", (guint) getpid ());
+  *copy = *error;
 
-  args[1] = (gchar*) prg_name;
-  args[2] = buf;
+  copy->message = g_strdup (error->message);
 
-  pid = fork ();
-  if (pid == 0)
-    {
-      stack_trace (args);
-      _exit (0);
-    }
-  else if (pid == (pid_t) -1)
-    {
-      perror ("unable to fork gdb");
-      return;
-    }
-  
-  while (glib_on_error_halt)
-    ;
-  glib_on_error_halt = TRUE;
-#else
-  abort ();
-#endif
+  return copy;
 }
 
-static gboolean stack_trace_done = FALSE;
-
-static void
-stack_trace_sigchld (int signum)
+gboolean
+g_error_matches (const GError *error,
+                 GQuark        domain,
+                 gint          code)
 {
-  stack_trace_done = TRUE;
+  return error &&
+    error->domain == domain &&
+    error->code == code;
 }
 
-static void
-stack_trace (char **args)
+void
+g_set_error (GError     **err,
+             GQuark       domain,
+             gint         code,
+             const gchar *format,
+             ...)
 {
-#ifdef G_OS_UNIX
-  pid_t pid;
-  int in_fd[2];
-  int out_fd[2];
-  SELECT_MASK fdset;
-  SELECT_MASK readset;
-  struct timeval tv;
-  int sel, index, state;
-  char buffer[256];
-  char c;
-
-  stack_trace_done = FALSE;
-  signal (SIGCHLD, stack_trace_sigchld);
-
-  if ((pipe (in_fd) == -1) || (pipe (out_fd) == -1))
-    {
-      perror ("unable to open pipe");
-      _exit (0);
-    }
-
-  pid = fork ();
-  if (pid == 0)
-    {
-      close (0); dup (in_fd[0]);   /* set the stdin to the in pipe */
-      close (1); dup (out_fd[1]);  /* set the stdout to the out pipe */
-      close (2); dup (out_fd[1]);  /* set the stderr to the out pipe */
+  va_list args;
 
-      execvp (args[0], args);      /* exec gdb */
-      perror ("exec failed");
-      _exit (0);
-    }
-  else if (pid == (pid_t) -1)
-    {
-      perror ("unable to fork");
-      _exit (0);
-    }
-
-  FD_ZERO (&fdset);
-  FD_SET (out_fd[0], &fdset);
-
-  write (in_fd[1], "backtrace\n", 10);
-  write (in_fd[1], "p x = 0\n", 8);
-  write (in_fd[1], "quit\n", 5);
+  if (err == NULL)
+    return;
 
-  index = 0;
-  state = 0;
+  if (*err != NULL)
+    g_warning ("GError set over the top of a previous GError or uninitialized memory.\n"
+               "This indicates a bug in someone's code. You must ensure an error is NULL before it's set.");
+  
+  va_start (args, format);
+  *err = g_error_new_valist (domain, code, format, args);
+  va_end (args);
+}
 
-  while (1)
+void
+g_clear_error (GError **err)
+{
+  if (err && *err)
     {
-      readset = fdset;
-      tv.tv_sec = 1;
-      tv.tv_usec = 0;
-
-      sel = select (FD_SETSIZE, &readset, NULL, NULL, &tv);
-      if (sel == -1)
-        break;
-
-      if ((sel > 0) && (FD_ISSET (out_fd[0], &readset)))
-        {
-          if (read (out_fd[0], &c, 1))
-            {
-              switch (state)
-                {
-                case 0:
-                  if (c == '#')
-                    {
-                      state = 1;
-                      index = 0;
-                      buffer[index++] = c;
-                    }
-                  break;
-                case 1:
-                  buffer[index++] = c;
-                  if ((c == '\n') || (c == '\r'))
-                    {
-                      buffer[index] = 0;
-                      fprintf (stdout, "%s", buffer);
-                      state = 0;
-                      index = 0;
-                    }
-                  break;
-                default:
-                  break;
-                }
-            }
-        }
-      else if (stack_trace_done)
-        break;
+      g_error_free (*err);
+      *err = NULL;
     }
-
-  close (in_fd[0]);
-  close (in_fd[1]);
-  close (out_fd[0]);
-  close (out_fd[1]);
-  _exit (0);
-#else
-  abort ();
-#endif
 }
diff --git a/glib/gerror.h b/glib/gerror.h
new file mode 100644 (file)
index 0000000..a3a07d4
--- /dev/null
@@ -0,0 +1,72 @@
+/* gerror.h - Error reporting system
+ *
+ *  Copyright 2000 Red Hat, Inc.
+ *
+ * The Gnome Library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public License as
+ * published by the Free Software Foundation; either version 2 of the
+ * License, or (at your option) any later version.
+ *
+ * The Gnome Library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License along with the Gnome Library; see the file COPYING.LIB.  If not,
+ * write to the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ *   Boston, MA 02111-1307, USA.
+ */
+
+#ifndef __GERROR_H__
+#define __GERROR_H__
+
+#ifdef __cplusplus
+extern "C"
+{
+#endif
+
+typedef struct _GError GError;
+
+struct _GError
+{
+  GQuark       domain;
+  gint         code;
+  gchar       *message;
+};
+
+GError*  g_error_new           (GQuark         domain,
+                                gint           code,
+                                const gchar   *format,
+                                ...) G_GNUC_PRINTF (3, 4);
+
+GError*  g_error_new_literal   (GQuark         domain,
+                                gint           code,
+                                const gchar   *message);
+
+void     g_error_free          (GError        *error);
+GError*  g_error_copy          (const GError  *error);
+
+gboolean g_error_matches       (const GError  *error,
+                                GQuark         domain,
+                                gint           code);
+
+/* if (err) *err = g_error_new(domain, code, format, ...), also has
+ * some sanity checks.
+ */
+void     g_set_error           (GError       **err,
+                                GQuark         domain,
+                                gint           code,
+                                const gchar   *format,
+                                ...) G_GNUC_PRINTF (4, 5);
+
+/* if (err && *err) { g_error_free(*err); *err = NULL; } */
+void     g_clear_error         (GError       **err);
+
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* __GERROR_H__ */
+
index d7ac6d5..31f1e1a 100644 (file)
@@ -3326,6 +3326,7 @@ guint           g_thread_pool_get_num_unused_threads (void);
 void            g_thread_pool_stop_unused_threads    (void);
 
 #include <gunicode.h>
+#include <gerror.h>
 
 #ifdef __cplusplus
 }