Merge pull request #11936 from OSGeo/backport-11935-to-release/3.10

[Backport release/3.10] Fix warnings when building against Poppler 25.03.00

Author: rouault

frmts/pdf/pdfdataset.cpp

@@ -171,7 +171,7 @@ class GDALPDFOutputDev : public SplashOutputDev
                                            invert, interpolate, inlineImg);
         else
         {
-            str->reset();
+            VSIPDFFileStream::resetNoCheckReturnValue(str);
             if (inlineImg)
             {
                 skipBytes(str, width, height, 1, 1);
@@ -209,7 +209,7 @@ class GDALPDFOutputDev : public SplashOutputDev
                                        interpolate, maskColors, inlineImg);
         else
         {
-            str->reset();
+            VSIPDFFileStream::resetNoCheckReturnValue(str);
             if (inlineImg)
             {
                 skipBytes(str, width, height, colorMap->getNumPixelComps(),

frmts/pdf/pdfio.cpp

@@ -260,7 +260,7 @@ void VSIPDFFileStream::reset()
 /************************************************************************/
 
 #if POPPLER_MAJOR_VERSION > 25 ||                                              \
-    (POPPLER_MAJOR_VERSION == 25 && POPPLER_MINOR_VERSION > 2)
+    (POPPLER_MAJOR_VERSION == 25 && POPPLER_MINOR_VERSION >= 3)
 bool VSIPDFFileStream::unfilteredReset()
 {
     return reset();

frmts/pdf/pdfio.h

@@ -55,8 +55,18 @@ class VSIPDFFileStream final : public BaseStream
     virtual void reset() override;
 #endif
 
+    static void resetNoCheckReturnValue(Stream *str)
+    {
+#if POPPLER_MAJOR_VERSION > 25 ||                                              \
+    (POPPLER_MAJOR_VERSION == 25 && POPPLER_MINOR_VERSION >= 2)
+        CPL_IGNORE_RET_VAL(str->reset());
+#else
+        str->reset();
+#endif
+    }
+
 #if POPPLER_MAJOR_VERSION > 25 ||                                              \
-    (POPPLER_MAJOR_VERSION == 25 && POPPLER_MINOR_VERSION > 2)
+    (POPPLER_MAJOR_VERSION == 25 && POPPLER_MINOR_VERSION >= 3)
     virtual bool unfilteredReset() override;
 #else
     virtual void unfilteredReset() override;

frmts/pdf/pdfobject.cpp

@@ -1409,7 +1409,13 @@ int64_t GDALPDFStreamPoppler::GetLength(int64_t nMaxSize)
     if (m_nLength >= 0)
         return m_nLength;
 
+#if POPPLER_MAJOR_VERSION > 25 ||                                              \
+    (POPPLER_MAJOR_VERSION == 25 && POPPLER_MINOR_VERSION >= 2)
+    if (!m_poStream->reset())
+        return 0;
+#else
     m_poStream->reset();
+#endif
     m_nLength = 0;
     unsigned char readBuf[4096];
     int readChars;
@@ -1477,7 +1483,13 @@ int64_t GDALPDFStreamPoppler::GetRawLength()
         return m_nRawLength;
 
     auto undecodeStream = m_poStream->getUndecodedStream();
+#if POPPLER_MAJOR_VERSION > 25 ||                                              \
+    (POPPLER_MAJOR_VERSION == 25 && POPPLER_MINOR_VERSION >= 2)
+    if (!undecodeStream->reset())
+        return 0;
+#else
     undecodeStream->reset();
+#endif
     m_nRawLength = 0;
     while (undecodeStream->getChar() != EOF)
         m_nRawLength++;