unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#45811: 28.0.50; [PATCH] Respect remember-save-after-remembering on remember-diary-extract-entries
@ 2021-01-12  4:21 Gabriel do Nascimento Ribeiro
  2021-01-12 14:26 ` Lars Ingebrigtsen
  2021-01-20 16:54 ` Lars Ingebrigtsen
  0 siblings, 2 replies; 6+ messages in thread
From: Gabriel do Nascimento Ribeiro @ 2021-01-12  4:21 UTC (permalink / raw)
  To: 45811

[-- Attachment #1: Type: text/plain, Size: 666 bytes --]

On remember.el, when option 'remember-handler-functions' contains
handler 'remember-diary-extract-entries', entries will be appended to
Diary file. However, the Diary file will not be saved automatically. I
could not find any documentation describing if this behavior is
expected, and the docstring for 'remember-save-after-remembering' does
not mention how the Diary file should be handled in this case. Given
that, there is a patch attached to automatically save the Diary file
when option 'remember-save-after-remembering' is 't'. If this is the
wrong behavior, we could improve the improve the docstring or show the
Diary buffer to user after data is appended.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Respect-remember-save-after-remembering-on-remember-.patch --]
[-- Type: text/x-diff, Size: 1058 bytes --]

From aa305cc45029387cba0de9ee00a4b58e1b022104 Mon Sep 17 00:00:00 2001
From: Gabriel do Nascimento Ribeiro <gabriel.nascimento@nubank.com.br>
Date: Tue, 12 Jan 2021 01:11:26 -0300
Subject: [PATCH] Respect remember-save-after-remembering on
 remember-diary-extract-entries

---
 lisp/textmodes/remember.el | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/lisp/textmodes/remember.el b/lisp/textmodes/remember.el
index 98d3a3856e..911523fb47 100644
--- a/lisp/textmodes/remember.el
+++ b/lisp/textmodes/remember.el
@@ -542,7 +542,10 @@ remember-diary-extract-entries
         (push (remember-diary-convert-entry (match-string 1)) list))
       (when list
         (diary-make-entry (mapconcat 'identity list "\n")
-                          nil remember-diary-file))
+                          nil remember-diary-file)
+        (when remember-save-after-remembering
+          (with-current-buffer (find-buffer-visiting diary-file)
+            (save-buffer))))
       nil))) ;; Continue processing
 
 ;;; Internal Functions:
-- 
2.27.0


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

end of thread, other threads:[~2021-01-20 16:54 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-12  4:21 bug#45811: 28.0.50; [PATCH] Respect remember-save-after-remembering on remember-diary-extract-entries Gabriel do Nascimento Ribeiro
2021-01-12 14:26 ` Lars Ingebrigtsen
2021-01-12 19:05   ` Gabriel do Nascimento Ribeiro
2021-01-12 19:26     ` Lars Ingebrigtsen
2021-01-20  0:12       ` Gabriel do Nascimento Ribeiro
2021-01-20 16:54 ` Lars Ingebrigtsen

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

	https://git.savannah.gnu.org/cgit/emacs.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).