Re: [O] [PATCH] Fix message format in org-notmuch-search-open

[Nicolas Goaziou]

Hello,

Christopher League lea...@contrapunctus.net writes:

 * org-notmuch.el (org-notmuch-search-open): Bug fix
 When opening a notmuch-search link, we use =message= to display the
 path at the bottom of the screen.  This would signal Not enough
 arguments for format string when the path contained %-signs, as it is
 likely to when the query contains spaces:
 [[notmuch-search:tag:inbox%2520not%2520tag:bulk%2520org]]

Applied. Thank you.


Regards,

-- 
Nicolas Goaziou

[O] [PATCH] Fix message format in org-notmuch-search-open

[Christopher League]

* org-notmuch.el (org-notmuch-search-open): Bug fix
When opening a notmuch-search link, we use =message= to display the
path at the bottom of the screen.  This would signal Not enough
arguments for format string when the path contained %-signs, as it is
likely to when the query contains spaces:
[[notmuch-search:tag:inbox%2520not%2520tag:bulk%2520org]]

That query appears to be double-escaped, which also might contribute
to the problem, but either way: we should use =(message %s str)= to
print arbitrary strings, not =(message str)=.
---
 contrib/lisp/org-notmuch.el | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/contrib/lisp/org-notmuch.el b/contrib/lisp/org-notmuch.el
index ae9b50b..712ec5a 100644
--- a/contrib/lisp/org-notmuch.el
+++ b/contrib/lisp/org-notmuch.el
@@ -113,7 +113,7 @@ Can link to more than one message, if so all matching 
messages are shown.
 
 (defun org-notmuch-search-open (path)
   Follow a notmuch message link specified by PATH.
-  (message path)
+  (message %s path)
   (funcall org-notmuch-search-open-function path))
 
 (defun org-notmuch-search-follow-link (search)
-- 
2.4.0