[poppler] [REMINDER] Poppler 0.6 release

[Krzysztof Kowalczyk]

On 8/25/07, Albert Astals Cid <aacid at kde.org> wrote:
> *** beign more lax about ending %%EOF ***
> Lately there seem to be some pdf around that do not end with %%EOF. That sucks
> as they break the check i introduced for %%EOF beign present and are not REAL
> pdf.
> This check is not really needed for "finished" files, but on kpdf/okular we
> have that handy "Check for file changes" that reloads the document once the
> file has changed (useful for latex people), the problem with that is that you
> know it changed, but not that it will not change anymore, so we often ended
> up reloading incomplete files that randomly crashed poppler.
> So i introduced the %%EOF check to avoid that.
> But it seems that is hurting us now, so we can remove the %%EOF check and try
> to improve poppler not to crash on incomplete documents or leave it that way
> and keep repeating our users that if it does not have an %%EOF it's not a PDF
> and that they should bug the producer not the consumer of the pdf file.
> Or any other suggestion?

I vote for removal of %%EOF check and making poppler not crash.

I've seen files with no proper %%EOF in my testing. Trying to educate
users of software that uses poppler that those are not valid PDF files
is a loosing proposition. If a file opens in Adobe Reader (and they
do) the user doesn't care why poppler can't handle it.

-- kjk