[pkgsrc/trunk]: pkgsrc/graphics/inkscape inkscape: fix build with poppler-0.65.0

[wiz <wiz%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/8730bdadeede
branches:  trunk
changeset: 309660:8730bdadeede
user:      wiz <wiz%pkgsrc.org@localhost>
date:      Thu Jun 21 10:25:35 2018 +0000

description:
inkscape: fix build with poppler-0.65.0

diffstat:

 graphics/inkscape/distinfo                                                     |    4 +-
 graphics/inkscape/patches/patch-src_extension_internal_pdfinput_pdf-parser.cpp |  103 +++++----
 2 files changed, 55 insertions(+), 52 deletions(-)

diffs (truncated from 464 to 300 lines):

diff -r 4eeb367721fb -r 8730bdadeede graphics/inkscape/distinfo
--- a/graphics/inkscape/distinfo        Thu Jun 21 09:34:50 2018 +0000
+++ b/graphics/inkscape/distinfo        Thu Jun 21 10:25:35 2018 +0000
@@ -1,4 +1,4 @@
-$NetBSD: distinfo,v 1.63 2018/05/01 06:17:11 wiz Exp $
+$NetBSD: distinfo,v 1.64 2018/06/21 10:25:35 wiz Exp $
 
 SHA1 (inkscape-0.92.2.tar.bz2) = c620d7aac594a0f839f2f7840d57e26f00991d16
 RMD160 (inkscape-0.92.2.tar.bz2) = 5508b38c47ed0a860a734eef8f8ca8e53f4e923a
@@ -13,7 +13,7 @@
 SHA1 (patch-config.h.cmake) = 1dcc7237fba3dbdfd7993ea2f5a2176785fb561e
 SHA1 (patch-src_CMakeLists.txt) = ccd5eeca81891f83c5eaf8cb80d33f001beaf55c
 SHA1 (patch-src_extension_internal_pdfinput_pdf-input.cpp) = 437cb3a461df047fef3c91e86c89cdcf15ef9965
-SHA1 (patch-src_extension_internal_pdfinput_pdf-parser.cpp) = 4ec25124dabf13ab3f68bcc626a2152ab3115a00
+SHA1 (patch-src_extension_internal_pdfinput_pdf-parser.cpp) = a15b784ff4941807c31958643d671961fff248ac
 SHA1 (patch-src_extension_internal_pdfinput_pdf-parser.h) = ba408f45544853f9323f8bd879bd55604072bb7a
 SHA1 (patch-src_extension_internal_pdfinput_svg-builder.cpp) = d7bfff0d3a1309b96eb70bc75b08d5d1ddf400da
 SHA1 (patch-src_io_inkjar.cpp) = 060d59225cfe7b66db5e9f41e76860df9ce12d01
diff -r 4eeb367721fb -r 8730bdadeede graphics/inkscape/patches/patch-src_extension_internal_pdfinput_pdf-parser.cpp
--- a/graphics/inkscape/patches/patch-src_extension_internal_pdfinput_pdf-parser.cpp    Thu Jun 21 09:34:50 2018 +0000
+++ b/graphics/inkscape/patches/patch-src_extension_internal_pdfinput_pdf-parser.cpp    Thu Jun 21 10:25:35 2018 +0000
@@ -1,4 +1,4 @@
-$NetBSD: patch-src_extension_internal_pdfinput_pdf-parser.cpp,v 1.7 2018/05/01 06:17:11 wiz Exp $
+$NetBSD: patch-src_extension_internal_pdfinput_pdf-parser.cpp,v 1.8 2018/06/21 10:25:35 wiz Exp $
 
 - Object.h is included in pdf-parser.h -- see patch for pdf-parser.h.
 - Support poppler 0.58
@@ -7,8 +7,11 @@
 
 --- src/extension/internal/pdfinput/pdf-parser.cpp.orig        2017-08-06 20:44:00.000000000 +0000
 +++ src/extension/internal/pdfinput/pdf-parser.cpp
-@@ -41,7 +41,7 @@ extern "C" {
- #include "goo/GooHash.h"
+@@ -38,10 +38,9 @@ extern "C" {
+ 
+ #include "goo/gmem.h"
+ #include "goo/GooTimer.h"
+-#include "goo/GooHash.h"
  #include "GlobalParams.h"
  #include "CharTypes.h"
 -#include "Object.h"
@@ -16,7 +19,7 @@
  #include "Array.h"
  #include "Dict.h"
  #include "Stream.h"
-@@ -414,13 +414,21 @@ void PdfParser::parse(Object *obj, GBool
+@@ -414,13 +413,21 @@ void PdfParser::parse(Object *obj, GBool
  
    if (obj->isArray()) {
      for (int i = 0; i < obj->arrayGetLength(); ++i) {
@@ -38,7 +41,7 @@
      }
    } else if (!obj->isStream()) {
        error(errInternal, -1, "Weird page contents");
-@@ -439,7 +447,11 @@ void PdfParser::go(GBool /*topLevel*/)
+@@ -439,7 +446,11 @@ void PdfParser::go(GBool /*topLevel*/)
  
    // scan a sequence of objects
    int numArgs = 0;
@@ -50,7 +53,7 @@
    while (!obj.isEOF()) {
  
      // got a command - execute it
-@@ -457,14 +469,20 @@ void PdfParser::go(GBool /*topLevel*/)
+@@ -457,14 +468,20 @@ void PdfParser::go(GBool /*topLevel*/)
        // Run the operation
        execOp(&obj, args, numArgs);
  
@@ -71,7 +74,7 @@
  
      // too many arguments - something is wrong
      } else {
-@@ -475,13 +493,21 @@ void PdfParser::go(GBool /*topLevel*/)
+@@ -475,13 +492,21 @@ void PdfParser::go(GBool /*topLevel*/)
        printf("\n");
        fflush(stdout);
        }
@@ -93,7 +96,7 @@
  
    // args at end with no command
    if (numArgs > 0) {
-@@ -495,8 +521,10 @@ void PdfParser::go(GBool /*topLevel*/)
+@@ -495,8 +520,10 @@ void PdfParser::go(GBool /*topLevel*/)
        printf("\n");
        fflush(stdout);
      }
@@ -104,7 +107,7 @@
    }
  }
  
-@@ -692,9 +720,13 @@ void PdfParser::opSetDash(Object args[],
+@@ -692,9 +719,13 @@ void PdfParser::opSetDash(Object args[],
    if (length != 0) {
      dash = (double *)gmallocn(length, sizeof(double));
      for (int i = 0; i < length; ++i) {
@@ -118,7 +121,7 @@
      }
    }
    state->setLineDash(dash, length, args[1].getNum());
-@@ -744,12 +776,18 @@ void PdfParser::opSetExtGState(Object ar
+@@ -744,12 +775,18 @@ void PdfParser::opSetExtGState(Object ar
    GBool haveBackdropColor = gFalse;
    GBool alpha = gFalse;
  
@@ -137,7 +140,7 @@
      return;
    }
    if (printCommands) {
-@@ -759,7 +797,11 @@ void PdfParser::opSetExtGState(Object ar
+@@ -759,7 +796,11 @@ void PdfParser::opSetExtGState(Object ar
    }
  
    // transparency support: blend mode, fill/stroke opacity
@@ -149,7 +152,7 @@
      GfxBlendMode mode = gfxBlendNormal;
      if (state->parseBlendMode(&obj2, &mode)) {
        state->setBlendMode(mode);
-@@ -767,40 +809,71 @@ void PdfParser::opSetExtGState(Object ar
+@@ -767,40 +808,71 @@ void PdfParser::opSetExtGState(Object ar
        error(errSyntaxError, getPos(), "Invalid blend mode in ExtGState");
      }
    }
@@ -221,7 +224,7 @@
    }
    if (obj2.isName(const_cast<char*>("Default")) ||
        obj2.isName(const_cast<char*>("Identity"))) {
-@@ -809,9 +882,15 @@ void PdfParser::opSetExtGState(Object ar
+@@ -809,9 +881,15 @@ void PdfParser::opSetExtGState(Object ar
    } else if (obj2.isArray() && obj2.arrayGetLength() == 4) {
      int pos = 4;
      for (int i = 0; i < 4; ++i) {
@@ -237,7 +240,7 @@
        if (!funcs[i]) {
        pos = i;
        break;
-@@ -828,21 +907,37 @@ void PdfParser::opSetExtGState(Object ar
+@@ -828,21 +906,37 @@ void PdfParser::opSetExtGState(Object ar
    } else if (!obj2.isNull()) {
      error(errSyntaxError, getPos(), "Invalid transfer function in ExtGState");
    }
@@ -275,7 +278,7 @@
        funcs[0] = Function::parse(&obj3);
        if (funcs[0]->getInputSize() != 1 ||
            funcs[0]->getOutputSize() != 1) {
-@@ -851,26 +946,45 @@ void PdfParser::opSetExtGState(Object ar
+@@ -851,26 +945,45 @@ void PdfParser::opSetExtGState(Object ar
          funcs[0] = NULL;
        }
        }
@@ -321,7 +324,7 @@
  #if defined(POPPLER_EVEN_NEWER_NEW_COLOR_SPACE_API)
            blendingColorSpace = GfxColorSpace::parse(NULL, &obj5, NULL, NULL);
  #elif defined(POPPLER_EVEN_NEWER_COLOR_SPACE_API)
-@@ -879,15 +993,25 @@ void PdfParser::opSetExtGState(Object ar
+@@ -879,15 +992,25 @@ void PdfParser::opSetExtGState(Object ar
            blendingColorSpace = GfxColorSpace::parse(&obj5, NULL);
  #endif
          }
@@ -347,7 +350,7 @@
          if (!haveBackdropColor) {
            if (blendingColorSpace) {
              blendingColorSpace->getDefaultColor(&backdropColor);
-@@ -906,18 +1030,24 @@ void PdfParser::opSetExtGState(Object ar
+@@ -906,18 +1029,24 @@ void PdfParser::opSetExtGState(Object ar
        } else {
          error(errSyntaxError, getPos(), "Invalid soft mask in ExtGState - missing group");
        }
@@ -372,7 +375,7 @@
  }
  
  void PdfParser::doSoftMask(Object *str, GBool alpha,
-@@ -938,43 +1068,79 @@ void PdfParser::doSoftMask(Object *str, 
+@@ -938,43 +1067,79 @@ void PdfParser::doSoftMask(Object *str, 
    dict = str->streamGetDict();
  
    // check form type
@@ -452,7 +455,7 @@
    resDict = obj1.isDict() ? obj1.getDict() : (Dict *)NULL;
  
    // draw it
-@@ -987,7 +1153,9 @@ void PdfParser::doSoftMask(Object *str, 
+@@ -987,7 +1152,9 @@ void PdfParser::doSoftMask(Object *str, 
    if (blendingColorSpace) {
      delete blendingColorSpace;
    }
@@ -462,7 +465,7 @@
  }
  
  void PdfParser::opSetRenderingIntent(Object /*args*/[], int /*numArgs*/)
-@@ -1084,7 +1252,11 @@ void PdfParser::opSetFillColorSpace(Obje
+@@ -1084,7 +1251,11 @@ void PdfParser::opSetFillColorSpace(Obje
    Object obj;
  
    state->setFillPattern(NULL);
@@ -474,7 +477,7 @@
  
    GfxColorSpace *colorSpace = 0;
  #if defined(POPPLER_EVEN_NEWER_NEW_COLOR_SPACE_API)
-@@ -1106,7 +1278,9 @@ void PdfParser::opSetFillColorSpace(Obje
+@@ -1106,7 +1277,9 @@ void PdfParser::opSetFillColorSpace(Obje
      colorSpace = GfxColorSpace::parse(&obj, NULL);
    }
  #endif
@@ -484,7 +487,7 @@
    if (colorSpace) {
    GfxColor color;
      state->setFillColorSpace(colorSpace);
-@@ -1125,7 +1299,11 @@ void PdfParser::opSetStrokeColorSpace(Ob
+@@ -1125,7 +1298,11 @@ void PdfParser::opSetStrokeColorSpace(Ob
    GfxColorSpace *colorSpace = 0;
  
    state->setStrokePattern(NULL);
@@ -496,7 +499,7 @@
  #if defined(POPPLER_EVEN_NEWER_NEW_COLOR_SPACE_API)
    if (obj.isNull()) {
      colorSpace = GfxColorSpace::parse(NULL, &args[0], NULL, NULL);
-@@ -1145,7 +1323,9 @@ void PdfParser::opSetStrokeColorSpace(Ob
+@@ -1145,7 +1322,9 @@ void PdfParser::opSetStrokeColorSpace(Ob
      colorSpace = GfxColorSpace::parse(&obj, NULL);
    }
  #endif
@@ -506,7 +509,7 @@
    if (colorSpace) {
      GfxColor color;
      state->setStrokeColorSpace(colorSpace);
-@@ -2310,7 +2490,7 @@ void PdfParser::opShowText(Object args[]
+@@ -2310,7 +2489,7 @@ void PdfParser::opShowText(Object args[]
      builder->updateFont(state);
      fontChanged = gFalse;
    }
@@ -515,7 +518,7 @@
  }
  
  // TODO not good that numArgs is ignored but args[] is used:
-@@ -2331,7 +2511,7 @@ void PdfParser::opMoveShowText(Object ar
+@@ -2331,7 +2510,7 @@ void PdfParser::opMoveShowText(Object ar
    ty = state->getLineY() - state->getLeading();
    state->textMoveTo(tx, ty);
    builder->updateTextPosition(tx, ty);
@@ -524,7 +527,7 @@
  }
  
  // TODO not good that numArgs is ignored but args[] is used:
-@@ -2354,7 +2534,7 @@ void PdfParser::opMoveSetShowText(Object
+@@ -2354,7 +2533,7 @@ void PdfParser::opMoveSetShowText(Object
    ty = state->getLineY() - state->getLeading();
    state->textMoveTo(tx, ty);
    builder->updateTextPosition(tx, ty);
@@ -533,7 +536,7 @@
  }
  
  // TODO not good that numArgs is ignored but args[] is used:
-@@ -2375,7 +2555,11 @@ void PdfParser::opShowSpaceText(Object a
+@@ -2375,7 +2554,11 @@ void PdfParser::opShowSpaceText(Object a
    wMode = state->getFont()->getWMode();
    a = args[0].getArray();
    for (int i = 0; i < a->getLength(); ++i) {
@@ -545,7 +548,7 @@
      if (obj.isNum()) {
        // this uses the absolute value of the font size to match
        // Acrobat's behavior
-@@ -2388,11 +2572,13 @@ void PdfParser::opShowSpaceText(Object a
+@@ -2388,11 +2571,13 @@ void PdfParser::opShowSpaceText(Object a
        }
        builder->updateTextShift(state, obj.getNum());
      } else if (obj.isString()) {
@@ -560,7 +563,7 @@
    }
  }
  
-@@ -2465,7 +2651,11 @@ void PdfParser::doShowText(GooString *s)
+@@ -2465,7 +2650,11 @@ void PdfParser::doShowText(GooString *s)
        //out->updateCTM(state, 1, 0, 0, 1, 0, 0);
        if (0){ /*!out->beginType3Char(state, curX + riseX, curY + riseY, tdx, tdy,
                               code, u, uLen)) {*/
@@ -572,7 +575,7 @@
        if ((resDict = ((Gfx8BitFont *)font)->getResources())) {
          pushResources(resDict);
        }
-@@ -2478,7 +2668,9 @@ void PdfParser::doShowText(GooString *s)
+@@ -2478,7 +2667,9 @@ void PdfParser::doShowText(GooString *s)
        if (resDict) {
          popResources();
        }
@@ -582,7 +585,7 @@
        }
        restoreState();
        // GfxState::restore() does *not* restore the current position,
-@@ -2540,24 +2732,44 @@ void PdfParser::opXObject(Object args[],
+@@ -2540,24 +2731,44 @@ void PdfParser::opXObject(Object args[],
  {
    Object obj1, obj2, obj3, refObj;
  
@@ -628,7 +631,7 @@
  /*    out->psXObject(obj1.getStream(),
                   obj3.isStream() ? obj3.getStream() : (Stream *)NULL);*/
    } else if (obj2.isName()) {
-@@ -2565,8 +2777,10 @@ void PdfParser::opXObject(Object args[],
+@@ -2565,8 +2776,10 @@ void PdfParser::opXObject(Object args[],
    } else {