[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Reducing the sit-for time in w3m-process-wait-process
- From: Naohiro Aota <naota@xxxxxxxxxx>
- Date: Fri, 12 Dec 2008 16:16:29 +0900
- X-ml-name: emacs-w3m
- X-mail-count: 10542
- References: <87vdu5tscu.fsf@xxxxxxxx> <8763m1j5xy.fsf@xxxxxxxx> <b4m4p1jauok.fsf@xxxxxxx> <kz3ah2u3la.fsf@xxxxxxxxxxxxxxxxxxxxx>
$B$;$C$+$/(B ChangeLog $B$b=q$$$F$$$?$@$-$^$7$?$7!"$H$j$"$($:(B commit $B$7$F$_$^(B
(BDavid Engster <deng@xxxxxxxxxxxxxxx> writes:
(B> Katsumi Yamaoka <yamaoka@xxxxxxx> writes:
(B>>>>>>> In [emacs-w3m : No.10494] David Engster wrote:
(B>>> David Engster <deng@xxxxxxxxxxxxxxx> writes:
(B>>>> I'm using the encapsulate-images feature from the shimbun package, and I
(B>>>> noticed that when an article contains many images, it takes pretty long
(B>>>> for w3m to fetch them. It seems to me this time is almost entirely spend
(B>>>> on the '(sit-for 1)' in w3m-process-wait-process, while the actual w3m
(B>>>> process is already finished. When I change this to '(sit-for 0.1)',
(B>>>> fetching the images is much faster. It also seems to me that this speeds
(B>>>> up refreshing shimbun groups in general.
(B>>> To followup on this: I sometimes got parsing errors with '(sit-for
(B>>> 0.1)', because the buffer didn't contain the whole output from the w3m
(B>>> binary. However, I could fix this by using accept-process-output in the
(B>>> w3m sentinel. Now everything works without problems, and the reduced
(B>>> sit-for time makes working with emacs-w3m faster. Before, a w3m call
(B>>> would at least take one second, even when w3m just received a redirect
(B>>> or was fetching a small picture for a shimbun article. I attached the
(B>>> two-lines patch to this mail.
(B>> Great! I verified that the patch makes shimbun fast beyond
(B>> recognition for fetching articles containing many images. David,
(B>> could you provide the ChangeLog entry?
(B> 2008-12-05 David Engster <dengste@xxxxxx>
(B> * w3m-proc.el (w3m-process-wait-process): Reduce sit-for time.
(B> (w3m-process-sentinel): Wait for process output (on Emacs only).
(B>> I'm not sure how to solve it but I'd like to install the patch very
(B>> much :) since I usually use Emacs, not XEmacs. Any idea? If no one
(B>> comments, I'll install it in the emacs-w3m CVS trunk with the
(B>> modification as follows:
(B> Looks good to me, but TSUCHIYA Masatoshi's concerns are certainly
(B> reasonable. I don't know how to really handle this, either.