emacs-orgmode@gnu.org archives
 help / color / mirror / code / Atom feed
* [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
@ 2012-04-05 14:25 Sebastien Vauban
  2012-04-06  7:40 ` Bastien
  2012-04-09 15:33 ` Bastien
  0 siblings, 2 replies; 11+ messages in thread
From: Sebastien Vauban @ 2012-04-05 14:25 UTC (permalink / raw)
  To: emacs-orgmode-mXXj517/zsQ

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

	* org-table.el (org-table-iterate-buffer-tables): Autoload function.

From 8bf259dea1d84dede8c193c2e8d8223a6e5e87ec Mon Sep 17 00:00:00 2001
From: Sebastien Vauban <sva-D0wtAvR13HarG/iDocfnWg@public.gmane.org>
Date: Thu, 5 Apr 2012 16:22:44 +0200
Subject: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables

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

diff --git a/lisp/org-table.el b/lisp/org-table.el
index c336f89..8dd449a 100644
--- a/lisp/org-table.el
+++ b/lisp/org-table.el
@@ -2963,6 +2963,7 @@ with the prefix ARG."
       (widen)
       (org-table-map-tables (lambda () (org-table-recalculate t)) t))))
 
+;;;###autoload
 (defun org-table-iterate-buffer-tables ()
   "Iterate all tables in the buffer, to converge inter-table dependencies."
  (interactive)
-- 
1.7.9

Best regards,
  Seb

-- 
Sebastien Vauban

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-05 14:25 [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables Sebastien Vauban
@ 2012-04-06  7:40 ` Bastien
  2012-04-06  8:40   ` Sebastien Vauban
  2012-04-09 15:33 ` Bastien
  1 sibling, 1 reply; 11+ messages in thread
From: Bastien @ 2012-04-06  7:40 UTC (permalink / raw)
  To: Sebastien Vauban; +Cc: emacs-orgmode

Hi Sébastien,

"Sebastien Vauban" <wxhgmqzgwmuf@spammotel.com> writes:

> 2012-04-05  Sebastien Vauban  <wxhgmqzgwmuf@spammotel.com>
>
> 	* org-table.el (org-table-iterate-buffer-tables): Autoload
> 	function.

Why do you need this function to be autoloaded?

Thanks,

-- 
 Bastien

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-06  7:40 ` Bastien
@ 2012-04-06  8:40   ` Sebastien Vauban
  2012-04-06  9:08     ` Bastien
  0 siblings, 1 reply; 11+ messages in thread
From: Sebastien Vauban @ 2012-04-06  8:40 UTC (permalink / raw)
  To: emacs-orgmode-mXXj517/zsQ

Hi Bastien,

Bastien wrote:
> "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org> writes:
>> 2012-04-05  Sebastien Vauban  <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org>
>>
>> 	* org-table.el (org-table-iterate-buffer-tables): Autoload
>> 	function.
>
> Why do you need this function to be autoloaded?

To be able to use it in batch mode without having to require explicitly
`org-table'.

Best regards,
  Seb

-- 
Sebastien Vauban

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-06  8:40   ` Sebastien Vauban
@ 2012-04-06  9:08     ` Bastien
  2012-04-06 10:09       ` Sebastien Vauban
  0 siblings, 1 reply; 11+ messages in thread
From: Bastien @ 2012-04-06  9:08 UTC (permalink / raw)
  To: Sebastien Vauban; +Cc: emacs-orgmode

Hi Sébastien,

"Sebastien Vauban" <wxhgmqzgwmuf@spammotel.com> writes:

> Hi Bastien,
>
> Bastien wrote:
>> "Sebastien Vauban" <wxhgmqzgwmuf@spammotel.com> writes:
>>> 2012-04-05  Sebastien Vauban  <wxhgmqzgwmuf@spammotel.com>
>>>
>>> 	* org-table.el (org-table-iterate-buffer-tables): Autoload
>>> 	function.
>>
>> Why do you need this function to be autoloaded?
>
> To be able to use it in batch mode without having to require explicitly
> `org-table'.

The problem is there are quite a lot of commands (interactive functions)
that can be used in batch mode.  Why adding autoload to this and not to 
another one?  

I don't want to open the door for one-by-one requests of this kind... 
so unless this is a more pressing need for an autoload cookie, I'll let
you (require 'org-table) in your script.

Does that make sense?

-- 
 Bastien

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-06  9:08     ` Bastien
@ 2012-04-06 10:09       ` Sebastien Vauban
  2012-04-09 11:04         ` Bastien
  0 siblings, 1 reply; 11+ messages in thread
From: Sebastien Vauban @ 2012-04-06 10:09 UTC (permalink / raw)
  To: emacs-orgmode-mXXj517/zsQ

Hi Bastien,

Bastien wrote:
> "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org> writes:
>> Bastien wrote:
>>> "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org> writes:
>>>> 2012-04-05  Sebastien Vauban  <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org>
>>>>
>>>> 	* org-table.el (org-table-iterate-buffer-tables): Autoload
>>>> 	function.
>>>
>>> Why do you need this function to be autoloaded?
>>
>> To be able to use it in batch mode without having to require explicitly
>> `org-table'.
>
> The problem is there are quite a lot of commands (interactive functions)
> that can be used in batch mode. Why adding autoload to this and not to
> another one?
>
> I don't want to open the door for one-by-one requests of this kind... so
> unless this is a more pressing need for an autoload cookie, I'll let you
> (require 'org-table) in your script.
>
> Does that make sense?

I'd be more in favor of trying to get autoload cookies for all the functions
which could need it, hence accepting such one-by-one patches -- which have no
border effects of any sort.

For example, you accepted the autoload cookie on org-version, a couple of
weeks ago.

But, OK, there is certainly not right or wrong position, so I respect yours.

Best regards,
  Seb

-- 
Sebastien Vauban

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-06 10:09       ` Sebastien Vauban
@ 2012-04-09 11:04         ` Bastien
  0 siblings, 0 replies; 11+ messages in thread
From: Bastien @ 2012-04-09 11:04 UTC (permalink / raw)
  To: Sebastien Vauban; +Cc: emacs-orgmode

Hi Sébastien,

"Sebastien Vauban" <wxhgmqzgwmuf@spammotel.com> writes:

> For example, you accepted the autoload cookie on org-version, a couple of
> weeks ago.

Because it seems quite obvious why people would need to check
org-version in batch mode.

> But, OK, there is certainly not right or wrong position, so I respect
> yours.

The core idea of my answer was this: please make a complete
list of commands that should be autoloaded then submit a single patch
for this.  Deciding one by one is definitely too time-consuming.

Thanks for your help in this area,

-- 
 Bastien

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-05 14:25 [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables Sebastien Vauban
  2012-04-06  7:40 ` Bastien
@ 2012-04-09 15:33 ` Bastien
  2012-04-18  9:19   ` Sebastien Vauban
  1 sibling, 1 reply; 11+ messages in thread
From: Bastien @ 2012-04-09 15:33 UTC (permalink / raw)
  To: Sebastien Vauban; +Cc: emacs-orgmode

Hi Sébastien,

"Sebastien Vauban" <wxhgmqzgwmuf@spammotel.com> writes:

> 2012-04-05  Sebastien Vauban  <wxhgmqzgwmuf@spammotel.com>
>
> 	* 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.

Thanks for bringing this up,

-- 
 Bastien

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-09 15:33 ` Bastien
@ 2012-04-18  9:19   ` Sebastien Vauban
  2012-04-20 11:25     ` Bastien
  0 siblings, 1 reply; 11+ messages in thread
From: Sebastien Vauban @ 2012-04-18  9:19 UTC (permalink / raw)
  To: emacs-orgmode-mXXj517/zsQ

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

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2012-04-18  9:19   ` Sebastien Vauban
@ 2012-04-20 11:25     ` Bastien
       [not found]       ` <87ehrizluq.fsf-whniv8GeeGkdnm+yROfE0A@public.gmane.org>
  0 siblings, 1 reply; 11+ messages in thread
From: Bastien @ 2012-04-20 11:25 UTC (permalink / raw)
  To: Sebastien Vauban; +Cc: emacs-orgmode

Hi Sébastien,

"Sebastien Vauban" <wxhgmqzgwmuf@spammotel.com> writes:

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

Applied against master, thanks.

> 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:

Thanks for the thorough check!

>From 3df73c03d9a8c56189cbe91ec752bcc3269536ca Mon Sep 17 00:00:00 2001
> From: Sebastien Vauban <sva@mygooglest.com>
> 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@spammotel.com>
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

You don't need to add the line above in your ChangeLog entry.

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

Thanks,

-- 
 Bastien

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
       [not found]       ` <87ehrizluq.fsf-whniv8GeeGkdnm+yROfE0A@public.gmane.org>
@ 2014-02-05  8:50         ` Sebastien Vauban
  2014-02-05  8:56           ` Bastien
  0 siblings, 1 reply; 11+ messages in thread
From: Sebastien Vauban @ 2014-02-05  8:50 UTC (permalink / raw)
  To: Bastien; +Cc: emacs-orgmode-mXXj517/zsQ

Hi Bastien,

Bastien wrote:
> "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org> writes:
>
>> In the same vein, I found `org-update-all-dblocks', for which I propose a
>> patch hereafter.
>
> Applied against master, thanks.
>
>> [...]
>
>> 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
>>
>> 	* org.el (org-update-all-dblocks): Autoload function.

When checking the status of some patches I provided, I just noticed that
the function `org-update-all-dblocks' is NOT autoloaded anymore.

Can you put the autoload cookie back?  Thanks.

Best regards,
  Seb

-- 
Sebastien Vauban

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables
  2014-02-05  8:50         ` Sebastien Vauban
@ 2014-02-05  8:56           ` Bastien
  0 siblings, 0 replies; 11+ messages in thread
From: Bastien @ 2014-02-05  8:56 UTC (permalink / raw)
  To: Sebastien Vauban; +Cc: public-emacs-orgmode-mXXj517/zsQ, Bastien



Hi Sébastien,

"Sebastien Vauban" <sva-news-D0wtAvR13HarG/iDocfnWg@public.gmane.org>
writes:

> When checking the status of some patches I provided, I just noticed that
> the function `org-update-all-dblocks' is NOT autoloaded anymore.
>
> Can you put the autoload cookie back?  Thanks.

Why is it necessary?

-- 
 Bastien

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2014-02-05  8:57 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-05 14:25 [PATCH] Add autoload cookie for function org-table-iterate-buffer-tables Sebastien Vauban
2012-04-06  7:40 ` Bastien
2012-04-06  8:40   ` Sebastien Vauban
2012-04-06  9:08     ` Bastien
2012-04-06 10:09       ` Sebastien Vauban
2012-04-09 11:04         ` Bastien
2012-04-09 15:33 ` Bastien
2012-04-18  9:19   ` Sebastien Vauban
2012-04-20 11:25     ` Bastien
     [not found]       ` <87ehrizluq.fsf-whniv8GeeGkdnm+yROfE0A@public.gmane.org>
2014-02-05  8:50         ` Sebastien Vauban
2014-02-05  8:56           ` Bastien

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).