Ticket #28 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Index broken with this pdf file

Reported by: LoneFox Assigned to: jordi
Priority: major Version:
Keywords: Cc:

Description

http://www.nologin.net/Downloads/Papers/reveng-memory-analysis.pdf
All page numbers are displayed as 0 and moving from one page to another does not work. The file works correctly in Acrobat Reader.

Change History

05/28/06 09:22:44 changed by jordi

Well... it works for me, both in SVN and version 0.1.4.

Is this the only file that doesn't work correctly?

05/28/06 13:02:19 changed by LoneFox

It seems to depend on poppler version: with 0.5.1 it works, with 0.5.2 it does not.

I tested several other files and this is the only one that is broken.

05/28/06 15:59:13 changed by jordi

  • status changed from new to assigned.

That is, since I'm using poppler 0.5.1 right now I didn't see it. I'll have to see what changed in poppler 0.5.2 and make it work with it.

06/05/06 13:11:47 changed by jordi

  • status changed from assigned to closed.
  • resolution set to fixed.

OK, I found that this PDF file uses named links instead of just page number. Poppler 0.5.1 and earlier didn't have support for them and just gave the page number, but in Poppler 0.5.2 and newer (at least 0.5.3, which I have tried) not. I have added support for named links in commit [203].

06/18/06 04:53:14 changed by LoneFox

  • status changed from closed to reopened.
  • version deleted.
  • resolution deleted.

The fix itself works, but this typo prevents it from being compiled in: 

diff -Nur epdfview-0.1.5-orig/configure.ac epdfview-0.1.5/configure.ac
--- epdfview-0.1.5-orig/configure.ac	2006-06-10 20:53:32.000000000 +0300
+++ epdfview-0.1.5/configure.ac	2006-06-18 11:42:03.000000000 +0300
@@ -40,7 +40,7 @@
 AC_SUBST([POPPLER_CFLAGS])
 AC_SUBST([POPPLER_LIBS])
 dnl Check if we have poppler version 0.5.2 or higher.
-PKG_CHECK_EXISTS([poppler-glib >= 0.5.2], [have_popppler_052=yes])
+PKG_CHECK_EXISTS([poppler-glib >= 0.5.2], [have_poppler_052=yes])
 if test "x$have_poppler_052" = "xyes"; then
     AC_DEFINE([HAVE_POPPLER_0_5_2], [1], [Define to 1 if you have Poppler version 0.5.2 or higher.])
 fi

06/18/06 05:23:41 changed by jordi

  • status changed from reopened to closed.
  • resolution set to fixed.

Oops, I didn't see it.

Thank you and sorry for that.