Attachment 92383 Details for Bug 73269 – new patch

[patch] new patch

p (text/plain), 1.88 KB, created by Christian Persch (GNOME) on 2014-01-19 12:21:48 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Christian Persch (GNOME)

Created: 2014-01-19 12:21:48 UTC

Size: 1.88 KB

patch

obsolete

>From 9d57318e4d23057c7ea640f5181d9c46a6db2872 Mon Sep 17 00:00:00 2001
>From: Christian Persch <chpe@gnome.org>
>Date: Fri, 3 Jan 2014 23:31:56 +0100
>Subject: [PATCH] glib: Install error callback
>
>Install an error callback so that poppler error messages can be redirected
>to the GLib logging API.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=73269
>---
> glib/poppler.cc | 52 ++++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 52 insertions(+)
>
>diff --git a/glib/poppler.cc b/glib/poppler.cc
>index cc2ca4d..ec294fa 100644
>--- a/glib/poppler.cc
>+++ b/glib/poppler.cc
>@@ -19,6 +19,10 @@
> #include <config.h>
> #include "poppler.h"
> 
>+#ifndef __GI_SCANNER__
>+#include <Error.h>
>+#endif
>+
> GQuark poppler_error_quark (void)
> {
>   static GQuark q = 0;
>@@ -56,3 +60,51 @@ poppler_get_version (void)
> {
>   return poppler_version;
> }
>+
>+#if  __GNUC__ > 2 || (__GNUC__ == 2 && __GNUC_MINOR__ >= 7)
>+
>+/* We want to install an error callback so that PDF syntax warnings etc
>+ * can be redirected through the GLib logging API instead of always just
>+ * going to stderr.
>+ */
>+
>+static void
>+error_cb (void *data G_GNUC_UNUSED,
>+          ErrorCategory category,
>+          Goffset pos,
>+          char *message)
>+{
>+  static const char * const cat_str[] = {
>+    "Syntax warning",
>+    "Syntax error",
>+    "",
>+    "",
>+    "",
>+    "Not permitted",
>+    "Unimplemented",
>+    "Internal"
>+  };
>+
>+  /* The following will never occur in poppler-glib */
>+  switch (category) {
>+  case errConfig:
>+  case errCommandLine:
>+  case errNotAllowed:
>+  default:
>+    return;
>+  }
>+
>+  g_log (G_LOG_DOMAIN, G_LOG_LEVEL_INFO,
>+         "%s at position %" G_GOFFSET_FORMAT ": %s",
>+         cat_str[category], (goffset) pos, message);
>+}
>+
>+static void poppler_constructor (void) __attribute__((__constructor__));
>+
>+static void
>+poppler_constructor (void)
>+{
>+  setErrorCallback (error_cb, NULL);
>+}
>+
>+#endif /* GNUC */
>-- 
>1.8.3.1
>

Actions: View | Splinter Review

Attachments on bug 73269: 91474 | 92383 | 92384 | 92386 | 92409

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.