Commits

Shlomi Fish committed 3f20dd8

Made sure if's are after "dSP;".

https://rt.cpan.org/Ticket/Display.html?id=69622 - thank to VKON.

  • Participants
  • Parent commits 133d611

Comments (0)

Files changed (2)

 Revision history for Perl extension XML::LibXML
 
+    - Moved some if blocks after the dSP; (which contains declarations) to be
+    compliant with C89/C90, which don't allow declarations in the middle of
+    a C function.
+        - resolves https://rt.cpan.org/Ticket/Display.html?id=69622
+        - thanks to Vadim / VKON.
+
 1.81            Sat Jul 16 18:30:02 IDT 2011
     - Add scripts/fast-eumm.pl to remove the explicit objects dependency on 
     the "Makefile" file so after running scripts/fast-eumm.pl one won't have to

File perl-libxml-sax.c

     xmlParserCtxtPtr ctxt = (xmlParserCtxtPtr)ctx;
     PmmSAXVectorPtr  sax  = (PmmSAXVectorPtr)ctxt->_private;
 
+    dTHX;
+    dSP;
+
     if (sax->joinchars)
     {
         PSaxCharactersFlush(ctxt, sax->charbuf);
     }
 
-    dTHX;
-    dSP;
 
     ENTER;
     SAVETMPS;
     SV * rv;
     SV * arv;
 
+    dSP;
+    
     if (sax->joinchars)
     {
         PSaxCharactersFlush(ctxt, sax->charbuf);
     }
 
-    dSP;
-    
     ENTER;
     SAVETMPS;
 
     SV * rv;
     HV * element;
 
+    dSP;
+
     if (sax->joinchars)
     {
         PSaxCharactersFlush(ctxt, sax->charbuf);
     }
 
-    dSP;
-
     ENTER;
     SAVETMPS;