Fix deprecation warnings in error handling.

Author: cfis

ext/libxml/ruby_xml_document.c

@@ -50,7 +50,7 @@ static VALUE rxml_error_get_handler(VALUE self)
   return block;
 }
 
-VALUE rxml_error_wrap(xmlErrorPtr xerror)
+VALUE rxml_error_wrap(const xmlError *xerror)
 {
   VALUE result = Qnil;
 
@@ -97,7 +97,7 @@ VALUE rxml_error_wrap(xmlErrorPtr xerror)
 }
 
 /* Hook that receives xml error message */
-static void structuredErrorFunc(void *userData, xmlErrorPtr xerror)
+static void structuredErrorFunc(void *userData, const xmlError *xerror)
 {
   VALUE error = rxml_error_wrap(xerror);
 
@@ -158,7 +158,7 @@ static VALUE rxml_error_reset_handler(VALUE self)
   return self;
 }
 
-NORETURN(void rxml_raise(xmlErrorPtr xerror))
+NORETURN(void rxml_raise(const xmlError *xerror))
 {
   /* Wrap error up as Ruby object and send it off to ruby */
   VALUE error = rxml_error_wrap(xerror);

ext/libxml/ruby_xml_dtd.c

@@ -1,14 +1,14 @@
-/* Please see the LICENSE file for copyright and distribution information */
-
-#ifndef __RXML_ERROR__
-#define __RXML_ERROR__
-
-#include <libxml/xmlerror.h>
-
-extern VALUE eXMLError;
-
-void rxml_init_error(void);
-VALUE rxml_error_wrap(xmlErrorPtr xerror);
-NORETURN(void rxml_raise(xmlErrorPtr xerror));
-
-#endif
+/* Please see the LICENSE file for copyright and distribution information */
+
+#ifndef __RXML_ERROR__
+#define __RXML_ERROR__
+
+#include <libxml/xmlerror.h>
+
+extern VALUE eXMLError;
+
+void rxml_init_error(void);
+VALUE rxml_error_wrap(const xmlError *xerror);
+NORETURN(void rxml_raise(const xmlError *xerror));
+
+#endif

ext/libxml/ruby_xml_error.c

@@ -156,7 +156,7 @@ static VALUE rxml_html_parser_context_file(int argc, VALUE* argv, VALUE klass)
 
   htmlParserCtxtPtr ctxt = htmlCreateFileParserCtxt(StringValuePtr(file), NULL);
   if (!ctxt)
-    rxml_raise(&xmlLastError);
+    rxml_raise(xmlGetLastError());
 
   /* This is annoying, but xmlInitParserCtxt (called indirectly above) and 
      xmlCtxtUseOptionsInternal (called below) initialize slightly different
@@ -197,7 +197,7 @@ static VALUE rxml_html_parser_context_io(int argc, VALUE* argv, VALUE klass)
   if (!ctxt)
   {
     xmlFreeParserInputBuffer(input);
-    rxml_raise(&xmlLastError);
+    rxml_raise(xmlGetLastError());
   }
 
   /* This is annoying, but xmlInitParserCtxt (called indirectly above) and 
@@ -212,7 +212,7 @@ static VALUE rxml_html_parser_context_io(int argc, VALUE* argv, VALUE klass)
   {
     xmlFreeParserInputBuffer(input);
     xmlFreeParserCtxt(ctxt);
-    rxml_raise(&xmlLastError);
+    rxml_raise(xmlGetLastError());
   }
   inputPush(ctxt, stream);
   result = rxml_html_parser_context_wrap(ctxt);
@@ -247,7 +247,7 @@ static VALUE rxml_html_parser_context_string(int argc, VALUE* argv, VALUE klass)
   ctxt = xmlCreateMemoryParserCtxt(StringValuePtr(string),
                                    (int)RSTRING_LEN(string));
   if (!ctxt)
-    rxml_raise(&xmlLastError);
+    rxml_raise(xmlGetLastError());
 
   /* This is annoying, but xmlInitParserCtxt (called indirectly above) and 
      xmlCtxtUseOptionsInternal (called below) initialize slightly different