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: /srv/bzr/emacs/trunk r110286: Sync Org 7.9.2 from the commit tagged "release_7.9.2" in Org's Git repo. Date: Wed, 03 Oct 2012 14:30:01 -0400 Message-ID: References: <87pq5267ew.fsf@bzg.ath.cx> <87626uuol5.fsf@Rainer.invalid> <5fobkm9jnu.fsf@fencepost.gnu.org> <87wqzat6n7.fsf@Rainer.invalid> <87wqz9vxmm.fsf@bzg.ath.cx> <87a9w4by89.fsf@Rainer.invalid> <87zk447pz9.fsf@bzg.ath.cx> <871uhgbwvz.fsf@Rainer.invalid> <87pq50ahk6.fsf@Rainer.invalid> <26txuchi1p.fsf@fencepost.gnu.org> <87ehlgagem.fsf@Rainer.invalid> <874nmbimni.fsf@Rainer.invalid> <87iparh25d.fsf@Rainer.invalid> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1349289045 2086 80.91.229.3 (3 Oct 2012 18:30:45 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Wed, 3 Oct 2012 18:30:45 +0000 (UTC) Cc: emacs-devel@gnu.org To: Achim Gratz Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Oct 03 20:30:50 2012 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1TJTiE-0001ld-0T for ged-emacs-devel@m.gmane.org; Wed, 03 Oct 2012 20:30:26 +0200 Original-Received: from localhost ([::1]:54218 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJTi8-0005hy-EC for ged-emacs-devel@m.gmane.org; Wed, 03 Oct 2012 14:30:20 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:43831) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJTi3-0005hL-SH for emacs-devel@gnu.org; Wed, 03 Oct 2012 14:30:19 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TJTi0-00064B-HD for emacs-devel@gnu.org; Wed, 03 Oct 2012 14:30:15 -0400 Original-Received: from ironport2-out.teksavvy.com ([206.248.154.182]:19446) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TJTi0-0005s8-Cz for emacs-devel@gnu.org; Wed, 03 Oct 2012 14:30:12 -0400 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: Av0EAG6Zu09FxLQG/2dsb2JhbABEtBGBCIIVAQEEAVYjBQsLDiYHCxQYDSSIHAW6CZBEA545hHqBWIMF X-IronPort-AV: E=Sophos;i="4.75,637,1330923600"; d="scan'208";a="200580963" Original-Received: from 69-196-180-6.dsl.teksavvy.com (HELO pastel.home) ([69.196.180.6]) by ironport2-out.teksavvy.com with ESMTP/TLS/ADH-AES256-SHA; 03 Oct 2012 14:30:01 -0400 Original-Received: by pastel.home (Postfix, from userid 20848) id 4A5E5594E8; Wed, 3 Oct 2012 14:30:01 -0400 (EDT) In-Reply-To: <87iparh25d.fsf@Rainer.invalid> (Achim Gratz's message of "Wed, 03 Oct 2012 19:54:38 +0200") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.2.50 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 206.248.154.182 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:154026 Archived-At: > I'd give it a shot, but this is all very delicately intertwined and it > appears that the documentation and the comments to gether either don't > tell the full story or are outdated in several places. I know, I spent a fair amount of time in that code a few years ago to eliminate an O(N^2) behavior and then later to add the support for file-local settings of generated-autoload-file. > Does anybody have better information on what the "public interfaces" > of that code are and what are the assumptions that a cleanup needs > to keep? IIRC there are fairly few entry points: update-directory-autoloads, update-file-autoloads, and batch-update-autoloads. Not sure about generate-file-autoloads which is either dead, or a very rarely used entry point. BTW, if/when considering how to fix it, it might be worth considering at the same time how to improve it so as to support ';;;###foo-autoload' directly (without having to scan the file twice, for example). Probably by looking for ";;;###\\(.*-\\)?autoload" and using an alist that maps from foo to the corresponding file. Stefan