From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org>
Subject: Re: [PATCH] Add autoload cookie for function
 org-table-iterate-buffer-tables
Date: Wed, 18 Apr 2012 11:19:45 +0200
Message-ID: <80vckx4csu.fsf@somewhere.org>
References: <80pqbm45nl.fsf@somewhere.org> <87hawshqd8.fsf@gnu.org>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org-mXXj517/zsQ@public.gmane.org>
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request-mXXj517/zsQ@public.gmane.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode-mXXj517/zsQ@public.gmane.org>
List-Help: <mailto:emacs-orgmode-request-mXXj517/zsQ@public.gmane.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request-mXXj517/zsQ@public.gmane.org?subject=subscribe>
Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org-mXXj517/zsQ@public.gmane.org
Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org-mXXj517/zsQ@public.gmane.org
To: emacs-orgmode-mXXj517/zsQ@public.gmane.org

Hi Bastien,

Bastien wrote:
> "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org> writes:
>>
>> 	* org-table.el (org-table-iterate-buffer-tables): Autoload
>> 	function.
>
> I finally accepted this in master. I reviewed org-table.el for other
> candidates and found `org-table-recalculate-buffer-tables', which is also
> autoloaded now.

I confirm the patch is OK. Thanks.

> Thanks for bringing this up,

In the same vein, I found `org-update-all-dblocks', for which I propose a
patch hereafter.

I've scanned the 121 occurrences of `(interactive)' in `org.el' to look for
extra potential candidates to the autoloading, but did not find any useful
ones. Hence, just this one to add:

>From 3df73c03d9a8c56189cbe91ec752bcc3269536ca Mon Sep 17 00:00:00 2001
From: Sebastien Vauban <sva-D0wtAvR13HarG/iDocfnWg@public.gmane.org>
Date: Wed, 18 Apr 2012 11:12:00 +0200
Subject: [PATCH 3/3] Add autoload cookie for org-update-all-dblocks

2012-04-18  Sebastien Vauban  <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org>

	* org.el (org-update-all-dblocks): Autoload function.

---
 lisp/org.el |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/lisp/org.el b/lisp/org.el
index 170ddc9..5473a20 100644
--- a/lisp/org.el
+++ b/lisp/org.el
@@ -11075,6 +11075,7 @@ Error if there is no such block at point."
       (goto-char pos)
       (error "Not in a dynamic block"))))
 
+;;;###autoload
 (defun org-update-all-dblocks ()
   "Update all dynamic blocks in the buffer.
 This function can be used in a hook."
-- 
1.7.9

Best regards,
  Seb

-- 
Sebastien Vauban