Bug: Org line commands and visual-line mode bindings [9.3.7 (9.3.7-15-gc9abb4-elpaplus @ /home/gustavo/.emacs.d/elpa/org-plus-contrib-20200727/)]

2020-07-27 Thread Gustavo Barros

Hi All,

The Org line commands -- `org-beginning-of-line', `org-end-of-line', and 
`org-kill-line' -- all take due care for the presence of 
`visual-line-mode' to do the right thing if it is turned on.  However, 
when `visual-line-mode' is indeed on, the bindings on 
`visual-line-mode-map' shadow Org's bindings, so that we actually get 
`beginning-of-visual-line', `end-of-visual-line', and `kill-visual-line' 
for the usual keybindings, instead of the much nicer specialized Org 
commands.


To check this, start with =emacs -Q=, set load-path to get the proper 
version of Org (as your case may be):


#+begin_src emacs-lisp
(add-to-list 'load-path "~/.emacs.d/elpa/org-plus-contrib-20200727")
#+end_src

Then visit an Org file, and enable "M-x visual-line-mode", and check the 
bindings with "C-h k C-a", "C-h k C-e", and "C-h k C-k" to get:


#+begin_example
beginning-of-visual-line
end-of-visual-line
kill-visual-line
#+end_example

I'm not sure this is a "bug", strictly speaking, or if it is correct 
unfortunate behavior.  Anyway, is there something that could be done 
from Org's side?


Best,
Gustavo.


Emacs  : GNU Emacs 27.0.91 (build 1, x86_64-pc-linux-gnu, GTK+ Version 
3.24.18, cairo version 1.16.0)

of 2020-06-29
Package: Org mode version 9.3.7 (9.3.7-15-gc9abb4-elpaplus @ 
/home/gustavo/.emacs.d/elpa/org-plus-contrib-20200727/)


current state:
==
(setq
org-src-mode-hook '(org-src-babel-configure-edit-buffer 
org-src-mode-configure-edit-buffer)

org-link-shell-confirm-function 'yes-or-no-p
org-metadown-hook '(org-babel-pop-to-session-maybe)
org-clock-out-hook '(org-clock-remove-empty-clock-drawer)
org-mode-hook '(#[0 "\300\301\302\303\304$\207" [add-hook 
change-major-mode-hook org-show-all append local] 5]

 #[0 "\300\301\302\303\304$\207"
		   [add-hook change-major-mode-hook 
		   org-babel-show-result-all append local] 5]
		 org-babel-result-hide-spec org-babel-hide-all-hashes 
		 org-eldoc-load)

org-archive-hook '(org-attach-archive-delete-maybe)
org-confirm-elisp-link-function 'yes-or-no-p
org-agenda-before-write-hook '(org-agenda-add-entry-text)
org-metaup-hook '(org-babel-load-in-session-maybe)
org-bibtex-headline-format-function #[257 "\300\236A\207" [:title] 3 
"\n\n(fn ENTRY)"]

org-babel-pre-tangle-hook '(save-buffer)
org-tab-first-hook '(org-babel-hide-result-toggle-maybe 
org-babel-header-arg-expand)
org-src-lang-modes '(("arduino" . arduino) ("redis" . redis) ("php" 
. php) ("C" . c) ("C++" . c++)
		  ("asymptote" . asy) ("bash" . sh) ("beamer" 
		  . latex) ("calc" . fundamental) ("cpp" . c++)
		  ("ditaa" . artist) ("dot" . fundamental) ("elisp" 
		  . emacs-lisp) ("ocaml" . tuareg)
		  ("screen" . shell-script) ("shell" . sh) ("sqlite" 
		  . sql))

org-occur-hook '(org-first-headline-recenter)
org-cycle-hook '(org-cycle-hide-archived-subtrees 
org-cycle-show-empty-lines

  org-optimize-window-after-visibility-change)
org-speed-command-hook '(org-speed-command-activate 
org-babel-speed-command-activate)

org-confirm-shell-link-function 'yes-or-no-p
org-link-parameters '(("attachment" :follow org-attach-open-link 
:export org-attach-export-link :complete

org-attach-complete-link)
		   ("id" :follow org-id-open) ("eww" :follow eww 
		   :store org-eww-store-link)
		   ("rmail" :follow org-rmail-open :store 
		   org-rmail-store-link)
		   ("mhe" :follow org-mhe-open :store 
		   org-mhe-store-link)
		   ("irc" :follow org-irc-visit :store 
		   org-irc-store-link :export org-irc-export)
		   ("info" :follow org-info-open :export 
		   org-info-export :store org-info-store-link)
		   ("gnus" :follow org-gnus-open :store 
		   org-gnus-store-link)
		   ("docview" :follow org-docview-open :export 
		   org-docview-export :store

org-docview-store-link)
		   ("bibtex" :follow org-bibtex-open :store 
		   org-bibtex-store-link)
		   ("bbdb" :follow org-bbdb-open :export 
		   org-bbdb-export :complete org-bbdb-complete-link

:store org-bbdb-store-link)
		   ("w3m" :store org-w3m-store-link) ("file+sys") 
		   ("file+emacs")

   ("shell" :follow org-link--open-shell)
		   ("news" :follow #[257 "\301\300\302Q!\207" 
		   ["news" browse-url ":"] 5 "\n\n(fn URL)"])
		   ("mailto" :follow #[257 "\301\300\302Q!\207" 
		   ["mailto" browse-url ":&q

[Help needed] Re: [Discuss] separate (recenter window-line) out of org-agenda-redo

2020-07-27 Thread numbch...@gmail.com
The `recenter` is useful for keeping position. I checked docstring of
`recenter`, have no clue how to implement a better solution. I also thinked
about ways like `with-current-buffer` etc. But can get around the
`recenter`.

Does anybody have any idea?

[stardiviner] GPG key ID: 47C32433
IRC(freeenode): stardiviner Twitter:  @numbchild
Key fingerprint = 9BAA 92BC CDDD B9EF 3B36  CB99 B8C4 B8E5 47C3 2433
Blog: http://stardiviner.github.io/


On Mon, Jul 27, 2020 at 3:46 PM Marco Wahl  wrote:

> "numbch...@gmail.com"  writes:
>
> [...]
>
> > I dive into source code of ~org-agenda-redo~ function.
> > Found this error is caused by ~(recenter window-line)~.
> >
> > I'm thinking what about to separate this code out? So function
> > ~org-agenda-redo~ can be used to non-interactive usage?
>
> My gut feeling says this is a good idea.
>
> Do you have a concrete implementation yet?
>
>
> Best regards,
> --
> Marco
>
>


Re: Waiting for refresh to finish...

2020-07-27 Thread tomas
On Mon, Jul 27, 2020 at 09:06:24PM +0200, Joseph Vidal-Rosset wrote:
> Le lun.  07/27/20 juil. 2020 à  08:28:03 , to...@tuxteam.de a  envoyé ce
> message:
> >  Try to remove that milkbox entry.
> >
> > Cheers
> > -- t
> >
> 
> I dit it immediately and it works now !
> 
> Many thanks Tomas ! You really just helped me very efficiently !

Glad it helped :)

Let's hope this value stays when you restart Emacs. If it came from a
prior customization (which is likely), saving your customization (which
you probably did) will do the trick. Otherwise, some more debugging is
in order.

Cheers
-- tomás


signature.asc
Description: Digital signature


Re: Waiting for refresh to finish...

2020-07-27 Thread Joseph Vidal-Rosset
Le lun.  07/27/20 juil. 2020 à  08:28:03 , to...@tuxteam.de a  envoyé ce
message:
>  Try to remove that milkbox entry.
>
> Cheers
> -- t
>

I dit it immediately and it works now !

Many thanks Tomas ! You really just helped me very efficiently !

Best wishes, 
-- 
Joseph 



Re: Waiting for refresh to finish...

2020-07-27 Thread tomas
On Mon, Jul 27, 2020 at 08:00:07PM +0200, Joseph Vidal-Rosset wrote:
> Hi again,
> 
> Many thanks for your reply. Here are the details requested. 
> 
> Le lun.  07/27/20 juil. 2020 à  07:27:42 , to...@tuxteam.de a  envoyé ce
> message:
> > On Mon, Jul 27, 2020 at 07:13:42PM +0200, Joseph Vidal-Rosset wrote:
> >> Thanks  for your  help,  Tomas,  here is  the  beginning  of my  init.el
> >> (suggestions to improve it are of course welcome):
> >
> > Caveat: as I said, I'm most probably not the right guy to answer that, but
> > hey.
> >
> > [...]
> >
> >> ;; Configure Emacs package manager. Not required anymore on Emacs > 27
> >> (if (version< emacs-version "27")
> >> (package-initialize)  )
> >> (require 'package)
> >> (setq package-archives
> >>   '(("melpa" . "https://melpa.org/packages/;)
> >> ("melpa-stable" . "https://stable.melpa.org/packages/;)
> >> ("org" . "https://orgmode.org/elpa/;)
> >>; ("gnu" . "https://elpa.gnu.org/packages/;)
> >>))
> >
> > This might be the relevant tidbit. But still, we don't know what version
> > your Emacs is, and thus whether this code is actually relevant or not.
> 
> joseph@mx:~$ emacs --version
> GNU Emacs 26.1

OK. So this variable is being read, probably (since 26.1 < 27), but...

> > What is the actual value of your variable `package-archives' [...]

> > Emacs   running   instance?  Try   doing   C-h   v  and   then   enter
> > `package-archives'.
> 
> package-archives’s value is
> (("melpa" . "https://melpa.org/packages/;)
>  ("melpa-stable" . "https://stable.melpa.org/packages/;)
>  ("org" . "https://orgmode.org/elpa/;)
>  ("melpa" . "http://melpa.milkbox.net/packages/;))
> Original value was 
> (("gnu" . "https://elpa.gnu.org/packages/;))

[...]

...but the actual variable's value has been "augmented" by a
(second) entry for "melpa", mapping to "http://melpa.milkbox.net/packages/;.

This one fails for me trying to resolve the name (host melpa.milkbox.net
is unknown to (at least my) DNS).

> > Are those sites reachable from your place? I.e. try something like

> joseph@mx:~$  curl -I https://stable.melpa.org/packages/
> HTTP/1.1 200 OK

[...]

Yes, this one works for me, too. But the milkbox one doesn't.
Now I have no idea what "update packages" will do in such a
case (and having two entries tagged with the same name looks
suspect too).

When you do the "describe variable" thing, there is a link in the
variable description "You can [customize] this variable" (actually
there are no square brackets there, it's my way to suggest that
this text looks different. Click on this link (or just put point
on it and hit ENTER) and you'll get a customization page. Try
to remove that milkbox entry.

Cheers
-- t


signature.asc
Description: Digital signature


Re: Waiting for refresh to finish...

2020-07-27 Thread Joseph Vidal-Rosset
Hi again,

Many thanks for your reply. Here are the details requested. 

Le lun.  07/27/20 juil. 2020 à  07:27:42 , to...@tuxteam.de a  envoyé ce
message:
> On Mon, Jul 27, 2020 at 07:13:42PM +0200, Joseph Vidal-Rosset wrote:
>> Thanks  for your  help,  Tomas,  here is  the  beginning  of my  init.el
>> (suggestions to improve it are of course welcome):
>
> Caveat: as I said, I'm most probably not the right guy to answer that, but
> hey.
>
> [...]
>
>> ;; Configure Emacs package manager. Not required anymore on Emacs > 27
>> (if (version< emacs-version "27")
>> (package-initialize)  )
>> (require 'package)
>> (setq package-archives
>>   '(("melpa" . "https://melpa.org/packages/;)
>> ("melpa-stable" . "https://stable.melpa.org/packages/;)
>> ("org" . "https://orgmode.org/elpa/;)
>>; ("gnu" . "https://elpa.gnu.org/packages/;)
>>  ))
>
> This might be the relevant tidbit. But still, we don't know what version
> your Emacs is, and thus whether this code is actually relevant or not.

joseph@mx:~$ emacs --version
GNU Emacs 26.1


>
> What is the actual value of your variable `package-archives' in your
> Emacs   running   instance?  Try   doing   C-h   v  and   then   enter
> `package-archives'.

package-archives’s value is
(("melpa" . "https://melpa.org/packages/;)
 ("melpa-stable" . "https://stable.melpa.org/packages/;)
 ("org" . "https://orgmode.org/elpa/;)
 ("melpa" . "http://melpa.milkbox.net/packages/;))
Original value was 
(("gnu" . "https://elpa.gnu.org/packages/;))

  This variable may be risky if used as a file-local variable.

Documentation:
An alist of archives from which to fetch.
The default value points to the GNU Emacs package repository.

Each element has the form (ID . LOCATION).
 ID is an archive name, as a string.
 LOCATION specifies the base location for the archive.
  If it starts with "http(s):", it is treated as an HTTP(S) URL;
  otherwise it should be an absolute directory name.
  (Other types of URL are currently not supported.)

Only add locations that you trust, since fetching and installing
a package can run arbitrary code.

You can customize this variable.

This variable was introduced, or its default value was changed, in
version 26.1 of Emacs.


> Are those sites reachable from your place? I.e. try something like
>
>   curl -I https://stable.melpa.org/packages/
> in a shell (likewise for the other URLs). For me, they are all four.
> We still don't know whether those are the ones your Emacs is trying
> to reach (and the hunch with networking is just that, a hunch).

joseph@mx:~$  curl -I https://stable.melpa.org/packages/
HTTP/1.1 200 OK
Server: nginx
Date: Mon, 27 Jul 2020 17:58:28 GMT
Content-Type: text/html; charset=utf-8
Content-Length: 306
Last-Modified: Fri, 14 Dec 2018 04:45:09 GMT
Connection: keep-alive
Vary: Accept-Encoding
ETag: "5c133555-132"
Strict-Transport-Security: max-age=15768000
Accept-Ranges: bytes


> Cheers
> -- t

Thanks for your kind help !

Jo. 

-- 
Joseph 



Re: Waiting for refresh to finish...

2020-07-27 Thread tomas
On Mon, Jul 27, 2020 at 07:13:42PM +0200, Joseph Vidal-Rosset wrote:
> Thanks  for your  help,  Tomas,  here is  the  beginning  of my  init.el
> (suggestions to improve it are of course welcome):

Caveat: as I said, I'm most probably not the right guy to answer that, but
hey.


[...]

> ;; Configure Emacs package manager. Not required anymore on Emacs > 27
> (if (version< emacs-version "27")
> (package-initialize)  )
> (require 'package)
> (setq package-archives
>   '(("melpa" . "https://melpa.org/packages/;)
> ("melpa-stable" . "https://stable.melpa.org/packages/;)
> ("org" . "https://orgmode.org/elpa/;)
>; ("gnu" . "https://elpa.gnu.org/packages/;)
>   ))

This might be the relevant tidbit. But still, we don't know what version
your Emacs is, and thus whether this code is actually relevant or not.

What is the actual value of your variable `package-archives' in your
Emacs running instance? Try doing C-h v and then enter `package-archives'.

Are those sites reachable from your place? I.e. try something like

  curl -I https://stable.melpa.org/packages/

in a shell (likewise for the other URLs). For me, they are all four.
We still don't know whether those are the ones your Emacs is trying
to reach (and the hunch with networking is just that, a hunch).

Cheers
-- t


signature.asc
Description: Digital signature


Re: Waiting for refresh to finish...

2020-07-27 Thread Joseph Vidal-Rosset
Thanks  for your  help,  Tomas,  here is  the  beginning  of my  init.el
(suggestions to improve it are of course welcome):

(defvar last-file-name-handler-alist file-name-handler-alist)
(setq gc-cons-threshold  602653184
  gc-cons-percentage 0.6
  file-name-handler-alist nil)

;;; General configuration section

;; Dumped Emacs
;; (when (boundp 'galactic-emacs-pdumper-dumped)
;;  ;; Restore `load-path'
;;  (setq load-path galactic-emacs-pdumper-load-path)
  ;; When Emacs starts from dump file, some default modes are not
  ;; enabled
;;  (global-font-lock-mode)
;;  (transient-mark-mode))

(when (> emacs-major-version 23)
   (require 'package)
  (package-initialize)
  (add-to-list 'package-archives
'("melpa" . "http://melpa.milkbox.net/packages/;)
 'APPEND))

;; Configure Emacs package manager. Not required anymore on Emacs > 27
(if (version< emacs-version "27")
(package-initialize)  )
(require 'package)
(setq package-archives
  '(("melpa" . "https://melpa.org/packages/;)
("melpa-stable" . "https://stable.melpa.org/packages/;)
("org" . "https://orgmode.org/elpa/;)
   ; ("gnu" . "https://elpa.gnu.org/packages/;)
))

;; Some combination of GNU TLS and Emacs fail to retrieve archive
;; contents over https.
;; 
https://www.reddit.com/r/emacs/comments/cdei4p/failed_to_download_gnu_archive_bad_request/etw48ux
;; https://debbugs.gnu.org/cgi/bugreport.cgi?bug=34341
(if (and (version< emacs-version "26.3") (>= libgnutls-version 30600))
(setq gnutls-algorithm-priority "NORMAL:-VERS-TLS1.3"))

;; Enable packages signature verification only if GPG is installed
(setq package-check-signature (when (executable-find "gpg") 'allow-unsigned))

;; Change the below priorities if you prefer melpa-stable packages.
;; Higher is better.
(setq package-archive-priorities
  '(
("melpa" .  4)
("melpa-stable" . 3)
("org" . 2)
("gnu" . 1)
)
   )

;; Bootstrap `use-package'
(unless (package-installed-p 'use-package)
  (package-refresh-contents)
  (package-install 'use-package))
(eval-when-compile
  (require 'use-package))
(use-package use-package-ensure-system-package
  :ensure t)
(setq package-enable-at-startup nil)


;; addendum from scimax

(require 'package)
(add-to-list 'package-archives
 '("melpa" . "http://melpa.milkbox.net/packages/;) t)
(package-initialize)



(when (not package-archive-contents)
  (package-refresh-contents))
-- 
Joseph 



Re: Waiting for refresh to finish...

2020-07-27 Thread tomas
On Mon, Jul 27, 2020 at 06:13:20PM +0200, Joseph Vidal-Rosset wrote:
> Hello everybody,
> 
> "Waiting for refresh to finish..." is the message that I get with U command
> after package-list-packages, but the refresh is never finished and I cannot
> upgrade the packages.

Hm. Doesn't happen here. Network problem?

I'm far from an expert in that area, but perhaps more details are needed.

What is the value of your variable `package-archives'? Are the URLs mentioned
there reachable from your neck of the woods?

Cheers
-- t


signature.asc
Description: Digital signature


Waiting for refresh to finish...

2020-07-27 Thread Joseph Vidal-Rosset
Hello everybody,

"Waiting for refresh to finish..." is the message that I get with U command
after package-list-packages, but the refresh is never finished and I cannot
upgrade the packages.

Help is welcome.

Best wishes,

Jo.


[PATCH] Fix recommendation in ORG-NEWS (was: Binding RET to org-return-and-maybe-indent)

2020-07-27 Thread Kévin Le Gouguec
Jarmo Hurri  writes:

> Kévin Le Gouguec  writes:
>
>> Can you tell me whether electric-indent-local-mode works better for
>> you?  If it does, I'll followup with a patch to ORG-NEWS.
>
> Seems to be working fine. Thank you very much.

Thanks for the confirmation.

Here is a patch for ORG-NEWS, then:

>From e5ed2be19d7ada3a0b6dd16fc220c4414b2af4e6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?K=C3=A9vin=20Le=20Gouguec?= 
Date: Mon, 27 Jul 2020 15:00:03 +0200
Subject: [PATCH] Fix recommendation in 9.4 release notes

Cf. .

Reported-by: Jarmo Hurri 

* etc/ORG-NEWS (=RET= and =C-j= now obey ~electric-indent-mode~):
Recommend disabling electric-indent-local-mode rather than
electric-indent-mode, as the latter impacts all buffers rather than
just the newly-created Org buffer.
---
 etc/ORG-NEWS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/etc/ORG-NEWS b/etc/ORG-NEWS
index bc93f8e4f..1ac7486a7 100644
--- a/etc/ORG-NEWS
+++ b/etc/ORG-NEWS
@@ -253,7 +253,7 @@ To get the previous behaviour back, disable ~electric-indent-mode~
 explicitly:
 
 #+begin_src emacs-lisp
-(add-hook 'org-mode-hook (lambda () (electric-indent-mode -1)))
+(add-hook 'org-mode-hook (lambda () (electric-indent-local-mode -1)))
 #+end_src
 
 *** New optional numeric argument for ~org-return~
-- 
2.27.0



diagnosing SRC blocks with different output on export

2020-07-27 Thread Alexandre Duret-Lutz
Hi,

Is there a way to mark certain SRC blocks such that if evaluating them
during export produces a result different from the one stored in the
file, then the export process aborts with an error?

I'm using org-mode to document an automaton library, and use org-babel
block to run some examples and have their output automatically
recomputed when the library is improved.  However, some examples
expect some specific automata to be produced, maybe because I discuss
some particularities of that example automaton in the text ; if an
update of the library causes this to change, I need to detect that
change to adjust all the text around it in the documentation, or find
a better exemple.   So I'd like org-mode to signal me that my expected
result is non-longuer what is produced by causing the export (and
therefore my build) to fail.

Can this be achieved already?

Reading https://orgmode.org/manual/Results-of-Evaluation.html it seems
like I'd like I'm asking for a new form of handling.  Something like
"compare" instead of "replace".

Best regards
-- 
Alexandre Duret-Lutz



Re: Repeating task hourly

2020-07-27 Thread Kevin Liu

> Is there any way to do this or are the docs out of date?

I made a few quick changes to org-habit and it works prima facie.  Will
continue testing for a bit.

>From e690d7b43143410d5d5af524bb1ff0819feebc24 Mon Sep 17 00:00:00 2001
From: nivekuil 
Date: Mon, 27 Jul 2020 04:37:32 -0700
Subject: [PATCH] org-habit: allow hourly repeat

---
 lisp/org-habit.el | 9 -
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/lisp/org-habit.el b/lisp/org-habit.el
index f76f0f213..b9bb729c1 100644
--- a/lisp/org-habit.el
+++ b/lisp/org-habit.el
@@ -157,11 +157,12 @@ means of creating calendar-based reminders."
   :group 'org-faces)
 
 (defun org-habit-duration-to-days (ts)
-  (if (string-match "\\([0-9]+\\)\\([dwmy]\\)" ts)
+  (if (string-match "\\([0-9]+\\)\\([hdwmy]\\)" ts)
   ;; lead time is specified.
   (floor (* (string-to-number (match-string 1 ts))
 		(cdr (assoc (match-string 2 ts)
-			'(("d" . 1)("w" . 7)
+			'(("h" . 0.041)
+			  ("d" . 1)("w" . 7)
 			  ("m" . 30.4) ("y" . 365.25))
 (error "Invalid duration string: %s" ts)))
 
@@ -199,9 +200,7 @@ This list represents a \"habit\" for the rest of this module."
   (setq sr-days (org-habit-duration-to-days scheduled-repeat)
 	sr-type (progn (string-match "[\\.+]?\\+" scheduled-repeat)
 			   (match-string-no-properties 0 scheduled-repeat)))
-  (unless (> sr-days 0)
-	(error "Habit %s scheduled repeat period is less than 1d" habit-entry))
-  (when (string-match "/\\([0-9]+[dwmy]\\)" scheduled-repeat)
+  (when (string-match "/\\([0-9]+[hdwmy]\\)" scheduled-repeat)
 	(setq dr-days (org-habit-duration-to-days
 		   (match-string-no-properties 1 scheduled-repeat)))
 	(if (<= dr-days sr-days)
-- 
2.27.0



Repeating task hourly

2020-07-27 Thread Kevin Liu
According to the manual at
https://orgmode.org/manual/Repeated-tasks.html hourly repeating tasks
are possible, but the code in org-habit.el seems to disagree, explicitly
disallowing an interval <1d:

(error "Habit %s scheduled repeat period is less than 1d" habit-entry))

Is there any way to do this or are the docs out of date?



Re: Binding RET to org-return-and-maybe-indent

2020-07-27 Thread Jarmo Hurri


Greetings Kévin.

Kévin Le Gouguec  writes:

> If I replace (electric-indent-mode -1) with
> (electric-indent-local-mode -1) in org-mode-hook, I get the behaviour
> we have with "Org 9.3" and "Org master, electric-indent-mode on".
>
> Can you tell me whether electric-indent-local-mode works better for
> you?  If it does, I'll followup with a patch to ORG-NEWS.

Seems to be working fine. Thank you very much.

All the best, and stay safe.

Jarmo




Re: [Discuss] separate (recenter window-line) out of org-agenda-redo

2020-07-27 Thread Marco Wahl
"numbch...@gmail.com"  writes:

[...]

> I dive into source code of ~org-agenda-redo~ function.
> Found this error is caused by ~(recenter window-line)~.
>
> I'm thinking what about to separate this code out? So function
> ~org-agenda-redo~ can be used to non-interactive usage?

My gut feeling says this is a good idea.

Do you have a concrete implementation yet?


Best regards,
-- 
Marco