From mboxrd@z Thu Jan 1 00:00:00 1970 Path: main.gmane.org!not-for-mail From: Richard Stallman Newsgroups: gmane.emacs.devel Subject: Re: recursive load case in openp Date: Thu, 11 Apr 2002 21:13:04 -0600 (MDT) Sender: emacs-devel-admin@gnu.org Message-ID: <200204120313.g3C3D4v22341@aztec.santafe.edu> References: <200204102343.IAA06716@etlken.m17n.org> <200204111301.g3BD1VI30043@rum.cs.yale.edu> Reply-To: rms@gnu.org NNTP-Posting-Host: localhost.gmane.org X-Trace: main.gmane.org 1018581336 11648 127.0.0.1 (12 Apr 2002 03:15:36 GMT) X-Complaints-To: usenet@main.gmane.org NNTP-Posting-Date: Fri, 12 Apr 2002 03:15:36 +0000 (UTC) Cc: handa@etl.go.jp, eliz@is.elta.co.il, walters@debian.org, emacs-devel@gnu.org Return-path: Original-Received: from quimby.gnus.org ([80.91.224.244]) by main.gmane.org with esmtp (Exim 3.33 #1 (Debian)) id 16vrXD-00031l-00 for ; Fri, 12 Apr 2002 05:15:35 +0200 Original-Received: from fencepost.gnu.org ([199.232.76.164]) by quimby.gnus.org with esmtp (Exim 3.12 #1 (Debian)) id 16vrnI-0002x4-00 for ; Fri, 12 Apr 2002 05:32:12 +0200 Original-Received: from localhost ([127.0.0.1] helo=fencepost.gnu.org) by fencepost.gnu.org with esmtp (Exim 3.34 #1 (Debian)) id 16vrX5-0001AI-00; Thu, 11 Apr 2002 23:15:27 -0400 Original-Received: from pele.santafe.edu ([192.12.12.119]) by fencepost.gnu.org with esmtp (Exim 3.34 #1 (Debian)) id 16vrUn-0000yB-00; Thu, 11 Apr 2002 23:13:05 -0400 Original-Received: from aztec.santafe.edu (aztec [192.12.12.49]) by pele.santafe.edu (8.11.6+Sun/8.9.3) with ESMTP id g3C3D4a15773; Thu, 11 Apr 2002 21:13:04 -0600 (MDT) Original-Received: (from rms@localhost) by aztec.santafe.edu (8.10.2+Sun/8.9.3) id g3C3D4v22341; Thu, 11 Apr 2002 21:13:04 -0600 (MDT) X-Authentication-Warning: aztec.santafe.edu: rms set sender to rms@aztec using -f Original-To: monnier+gnu/emacs@rum.cs.yale.edu In-Reply-To: <200204111301.g3BD1VI30043@rum.cs.yale.edu> (monnier+gnu/emacs@rum.cs.yale.edu) Errors-To: emacs-devel-admin@gnu.org X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.0.9 Precedence: bulk List-Help: List-Post: List-Subscribe: , List-Id: Emacs development discussions. List-Unsubscribe: , List-Archive: Xref: main.gmane.org gmane.emacs.devel:2561 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:2561 I would argue that the `run-hooks' part of the function should be taken out. That call is correct for the intended purpose of this function, which is to support with-output-to-temp-buffer. Anything which calls temp_output_buffer_setup and does not want to run that hook should not call temp_output_buffer_setup. coding.c and fileio.c should not call it. Of course, in the mean time we should just create temp_buffer_setup and have temp_output_buffer_setup call it. The actual code needed in coding.c was too small to be worth such a subroutine. I will fix fileio.c also.