GNU bug report logs - #8364
24.0.9999; PATCH: Fit to width/height/page for doc-view.el

Previous Next

Full log

View this message in rfc822 format

From: sho nakatani <lay.sakura <at> gmail.com>
To: Tassilo Horn <tassilo <at> member.fsf.org>
Cc: 8364 <at> debbugs.gnu.org
Subject: bug#8364: 24.0.9999; PATCH: Fit to width/height/page for doc-view.el
Date: Wed, 30 Mar 2011 19:43:21 +0900

[Message part 1 (text/plain, inline)]

Hi Tassilo,

> The problematic issue was that the `new-slice' calculated by the
> `mapcar' contained floating point numbers, which are not allowed for
> slice specs.  I found out by using `edebug-defun' on
> `doc-view-insert-image'.

Wow! Thank you for the report. I didn't realized.

But your function doesn't work for some files.
I attached an example PDF file (written in Japanse).

(If you cannot see the attachment, download it from:
 https://github.com/laysakura/TheoryOfComputation/raw/master/main.pdf )

> One minor thing: I would prefer if your three original functions would
> respect the current slice if there is one, instead of adding a fourth
> function for that purpose.  Could you please make it that way?

It's easy! I'm making a new function just for a test purpose.
So soon after we get perfectly working function, I'll combine this feature
to `doc-view-fit-(width|height|page)-to-window' .

> And I'm especially pleased that my assumption of calculating the right
> scales of both image and slices out of the plain images without
> resorting to external tools turned out being right. ;-)

That's because your code have rigid basic functions :-)

Anyway, please tell me the result of test for the attachment
and give me any advice.

--
Sho Nakatani

[main.pdf (application/pdf, attachment)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.