From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Stefan Monnier Newsgroups: gmane.emacs.devel Subject: Re: more url-utils? Date: Mon, 16 May 2011 12:45:03 -0300 Message-ID: References: <8739khju46.fsf@lifelogs.com> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1305560717 26526 80.91.229.12 (16 May 2011 15:45:17 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Mon, 16 May 2011 15:45:17 +0000 (UTC) Cc: Ted Zlatanov , emacs-devel@gnu.org To: Tom Tromey Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Mon May 16 17:45:12 2011 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([140.186.70.17]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1QLzyp-0007AS-EC for ged-emacs-devel@m.gmane.org; Mon, 16 May 2011 17:45:11 +0200 Original-Received: from localhost ([::1]:40380 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QLzyo-0000wY-WC for ged-emacs-devel@m.gmane.org; Mon, 16 May 2011 11:45:10 -0400 Original-Received: from eggs.gnu.org ([140.186.70.92]:38386) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QLzyl-0000uB-UB for emacs-devel@gnu.org; Mon, 16 May 2011 11:45:08 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QLzyl-0004yz-00 for emacs-devel@gnu.org; Mon, 16 May 2011 11:45:07 -0400 Original-Received: from fencepost.gnu.org ([140.186.70.10]:43068) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QLzyk-0004ys-Ui for emacs-devel@gnu.org; Mon, 16 May 2011 11:45:06 -0400 Original-Received: from 213-159-126-200.fibertel.com.ar ([200.126.159.213]:45235 helo=ceviche.home) by fencepost.gnu.org with esmtpsa (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1QLzyk-00047g-7z; Mon, 16 May 2011 11:45:06 -0400 Original-Received: by ceviche.home (Postfix, from userid 20848) id D756A66140; Mon, 16 May 2011 12:45:03 -0300 (ART) In-Reply-To: (Tom Tromey's message of "Mon, 16 May 2011 09:19:18 -0600") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 3) X-Received-From: 140.186.70.10 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:139428 Archived-At: Stefan> While Emacs uses them a lot, dynamically-scoped variables used as Stefan> implicit extra parameters are bad. In many cases, it's difficult to Stefan> avoid using them, but in this case, calling a function after the Stefan> url-retrieve call works just as well, so there's no excuse. > IIRC, in package.el I had to special-case some URL types, because > headers are inconsistently used. E.g., I think the 'ftp' method won't > leave headers in the buffer. It would be nice if this bit were handled > by the url package rather than all the callers. Sounds like an inconsistency that needs fixing, indeed. You might want to make a bug-report for it. Stefan PS: Of course, that doesn't mean that the way to distinguish the "with header" and "without header" cases should be via a dynamically-scoped variable.