From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Thierry Emery Newsgroups: gmane.emacs.help Subject: Re: Emacs 21 and w3 on Debian Date: Thu, 26 May 2005 12:11:03 +0200 Organization: Home, France Message-ID: <87zmui8fgo.fsf@spamfoil.invalid> References: <87sm0fbjx4.fsf@tiger.rapttech.com.au> <874qcvqar0.fsf@spamfoil.invalid> <87oeb2c2x5.fsf@tiger.rapttech.com.au> <87fywebcp7.fsf@tiger.rapttech.com.au> <87ll66pavk.fsf@spamfoil.invalid> <871x7vtdtq.fsf@spamfoil.invalid> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII X-Trace: sea.gmane.org 1117102501 4049 80.91.229.2 (26 May 2005 10:15:01 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Thu, 26 May 2005 10:15:01 +0000 (UTC) Original-X-From: help-gnu-emacs-bounces+geh-help-gnu-emacs=m.gmane.org@gnu.org Thu May 26 12:15:00 2005 Return-path: Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1DbFMR-0006Ad-UM for geh-help-gnu-emacs@m.gmane.org; Thu, 26 May 2005 12:13:08 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DbFQU-0008UJ-Lk for geh-help-gnu-emacs@m.gmane.org; Thu, 26 May 2005 06:17:18 -0400 Original-Path: shelby.stanford.edu!newsfeed.stanford.edu!postnews.google.com!news2.google.com!newsread.com!news-xfer.newsread.com!news-feed01.roc.ny.frontiernet.net!nntp.frontiernet.net!newscon06.news.prodigy.com!prodigy.net!border1.nntp.dca.giganews.com!nntp.giganews.com!newsfeed00.sul.t-online.de!newsfeed01.sul.t-online.de!t-online.de!newsfeed.stueberl.de!news.cs.univ-paris8.fr!proxad.net!infeed-1.proxad.net!news1-e.free.fr!not-for-mail Original-Newsgroups: gnu.emacs.help User-Agent: T-gnus/6.17.3 (based on No Gnus v0.3) SEMI/1.14.6 (Maruoka)FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 Emacs/22.0.50 (gnu/linux) MULE/5.0 (SAKAKI) Cancel-Lock: sha1:C/kInmfZkP6bGfkxdyd4NmBHgM8= Original-Lines: 47 Original-NNTP-Posting-Date: 26 May 2005 12:10:34 MEST Original-NNTP-Posting-Host: 82.233.241.49 Original-X-Trace: 1117102234 news1-e.free.fr 10027 82.233.241.49:56041 Original-X-Complaints-To: abuse@proxad.net Original-Xref: shelby.stanford.edu gnu.emacs.help:131542 Original-To: help-gnu-emacs@gnu.org X-BeenThere: help-gnu-emacs@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Users list for the GNU Emacs text editor List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: help-gnu-emacs-bounces+geh-help-gnu-emacs=m.gmane.org@gnu.org Errors-To: help-gnu-emacs-bounces+geh-help-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.help:27001 X-Report-Spam: http://spam.gmane.org/gmane.emacs.help:27001 I wrote: > Klaus Straubinger writes: > >> Wouldn't it be more correct to replace the first argument of >> url-callback-arguments with the redirect-uri if there are any >> such callback arguments present? I have experienced problems >> with your solution that went away with this improvement. > > Through looking at the code, i thought that i had chosen the most > cautious alternative, but it seems that i was wrong ... Ok, i have taken a much better look at the different places in Debian `sid' GNU Emacs 21.4 in which there are calls to `url-retrieve' and found out that my bogus patch caused problems in `w3-maybe-start-image-download' and `w3-maybe-start-background-image-download', but also in the Gnus library "nnweb.el". I have submitted two patches for these problems to Debian: - the first one replaces in "url-http.el" my erroneous: (list redirect-uri) with: (cons redirect-uri (and url-callback-arguments (cdr url-callback-arguments))) - the second patch adds in "nnweb.el" `url' as first argument for `nnweb-callback' and as third argument in the call to `nnweb-url-retrieve-asynch' in `nnweb-fetch-url' (arguments used when calling `nnweb-callback' start with this third argument). I have also submitted a modified patch to "url-http.el" as a follow-up to my M-x report-emacs-bug. The patch on "nnweb.el" is not applicable to CVS Emacs: library "nnweb.el" does not call `url-retrieve' anymore, but uses `mm-url-insert' instead, which calls `url-insert-file-contents'. Finally, i have found that backporting to Emacs 21.4 the new `url-retrieve-synchronously' from "url.el" in CVS Emacs avoids some cases of Emacs 21.4 locking up (e.g. when a site has lots of images), and have also submitted a patch for this to Debian. Thanks again for pointing out my mistake, Thierry -- thierry |dot| emery |at| free |dot| fr