[Poppler-bugs] [Bug 62735] Should not try to fopen a file we already have open

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Mon Apr 1 12:52:00 PDT 2013


https://bugs.freedesktop.org/show_bug.cgi?id=62735

Adam Reichold <adamreichold at myopera.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #77122|0                           |1
        is obsolete|                            |

--- Comment #13 from Adam Reichold <adamreichold at myopera.com> ---
Created attachment 77289
  --> https://bugs.freedesktop.org/attachment.cgi?id=77289&action=edit
add a GooFile class to encapsulate file read access using offsets

Hopefully, I have made the necessary additions for autotools and CMake to
generate the 'pread64' and 'lseek64' definitions. I also removed the - as far
as I can see - by now superfluous calls to 'stat' where file streams are
constructed since we can just use 'GooFile::size'.

@Albert: I will run regtest on this version overnight (looking good so far),
but I think this is basically ready for review.

@Thomas: I used Poppler's default regtest which AFAIK is non-threaded since the
default pdftoppm is. I did do a bit of smoke testing using qpdfview and
multi-threading.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20130401/410fd296/attachment.html>


More information about the Poppler-bugs mailing list