Re: [Bug] Force creating of Mail/sent on sending?

2013-08-26 Thread Bastien
Bastien bzg-whniv8geegkdnm+yrof...@public.gmane.org writes: Is there a way to avoid notmuch asking this? Found it: (setq notmuch-fcc-dirs nil) Sorry for the noise, -- Bastien ___ notmuch mailing list notmuch@notmuchmail.org

[Patch (rebased) 1/2] contrib: pick: remove unneeded variable notmuch-pick-buffer-name

2013-08-26 Thread Mark Walters
This variable is essentially unused: it was only used for making sure it itself got reset after a refresh of the buffer. It did this by passing an unnecessary argument to notmuch-pick-worker so remove that too. --- contrib/notmuch-pick/notmuch-pick.el | 15 --- 1 files changed, 4

[Patch v4 2/3] contrib: pick: thread tagging (including archiving) implemented

2013-08-26 Thread Mark Walters
Previously pick had no actions based on the entire thread: this adds some. Note in this version '*' is bound to `tag thread' which is not consistent with search or show. However it still might be the most natural thing (as it is similar to running * in the show pane). ---

[Patch v4 3/3] contrib: pick: bind M-p and M-n to prev/next thread

2013-08-26 Thread Mark Walters
This adds functions to go to the previous or next thread to pick. Prev-thread behaves similarly to prev-message in show: if you are on the top line of a thread it will go to the top of the previous thread, otherwise it will go to the top of the current thread. Next thread will always go to the top

[Patch (rebased) 0/2] contrib: pick: remove an unused variable

2013-08-26 Thread Mark Walters
This is a rebased version of id:1372712861-13515-1-git-send-email-markwalters1...@gmail.com. When documenting notmuch-pick-worker I realised that notmuch-pick-buffer-name was an essentially unused argument: it is only used to reset itself on refresh. (Possibly this was a mis-copy from

[Patch v4 0/3] Add some thread based actions to pick

2013-08-26 Thread Mark Walters
This is a rebased version of id:1371195472-441-1-git-send-email-markwalters1...@gmail.com The only other change is a new patch 3 adding previous/next thread commands: they are both very simple functions based on the first two patches. Best wishes Mark Mark Walters (3): contrib: pick: add

[Patch (rebased) 2/2] contrib: pick: add docstring for notmuch-pick-worker

2013-08-26 Thread Mark Walters
--- contrib/notmuch-pick/notmuch-pick.el |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index f6710e9..30757c1 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++

Re: [Patch (rebased) 0/2] contrib: pick: remove an unused variable

2013-08-26 Thread Tomi Ollila
On Sun, Aug 25 2013, Mark Walters markwalters1...@gmail.com wrote: This is a rebased version of id:1372712861-13515-1-git-send-email-markwalters1...@gmail.com. When documenting notmuch-pick-worker I realised that notmuch-pick-buffer-name was an essentially unused argument: it is only used

Re: [PATCH] lib: fix clang build

2013-08-26 Thread Tomi Ollila
On Sun, Aug 18 2013, Jani Nikula j...@nikula.org wrote: On Sun, 18 Aug 2013, Tomi Ollila tomi.oll...@iki.fi wrote: -typedef struct _notmuch_string_list notmuch_string_list_t; - anyway, would typedef struct visible _notmuch_string_list notmuch_string_list_t; have helped here (if yes is

Re: [PATCH 1/1] emacs: add '?' to some prefix keymaps to describe its bindings

2013-08-26 Thread Mark Walters
On Sat, 24 Aug 2013, Tomi Ollila tomi.oll...@iki.fi wrote: Added (private) function `notmuch--prefix-bindings' which wraps `describe-prefix-bindings' to provide more suitable user interface for showing the key sequences that follow a particular notmuch prefix keymap invocation. The key '?'

[PATCH 2/2] contrib: pick: remove sync tests

2013-08-26 Thread Mark Walters
Since the previous commit removed the sync parser we may as well remove its tests too. --- contrib/notmuch-pick/run-tests.sh |2 +- contrib/notmuch-pick/test/emacs-pick-sync | 64 - 2 files changed, 1 insertions(+), 65 deletions(-) delete mode 100755

[PATCH 0/2] contrib: pick: remove sync parser

2013-08-26 Thread Mark Walters
Pick has two possible parsing methods: a synchronous one (like show) and an async one (like search). The async parser has been the default since at least the time pick first entered contrib. I don't believe there have been any reported problems with it during this time so we might as well remove

[PATCH 1/2] contrib: pick: remove sync parser

2013-08-26 Thread Mark Walters
Previously pick had the option of using an async parser like search or a sync parser like show. The async parser has always been the default and it seems fine so we can remove the sync one and the corresponding defcustom. --- contrib/notmuch-pick/notmuch-pick.el | 36

Re: [PATCH 0/2] contrib: pick: remove sync parser

2013-08-26 Thread Tomi Ollila
On Mon, Aug 26 2013, Mark Walters markwalters1...@gmail.com wrote: Pick has two possible parsing methods: a synchronous one (like show) and an async one (like search). The async parser has been the default since at least the time pick first entered contrib. I don't believe there have been any

[PATCH 1/1] emacs: removed 3 duplicate functions from notmuch-show.el

2013-08-26 Thread Tomi Ollila
notmuch-show.el and notmuch.el had 3 duplicate, identical functions: notmuch-foreach-mime-part, notmuch-count-attachments and notmuch-save-attachments. Now these functions in notmuch-show.el are replaced with declare-functions pointing to notmuch(.el). --- emacs/notmuch-show.el | 39

[Patch v2 0/3] contrib: pick: remove an unused variable

2013-08-26 Thread Mark Walters
v2 of this series is at id:1377460214-4795-1-git-send-email-markwalters1...@gmail.com v2 had a bug on refresh view (which I should have tested more). The main pick view only worked by fluke as the initial call to pick-worker was inside a let binding. This version fixes the bug, moves the call to

[Patch v2 1/3] contrib: pick: remove unneeded variable notmuch-pick-buffer-name

2013-08-26 Thread Mark Walters
This variable is essentially unused: it was only used for making sure it itself got reset after a refresh of the buffer. It did this by passing an unnecessary argument to notmuch-pick-worker so remove that too. --- contrib/notmuch-pick/notmuch-pick.el | 25 + 1 files

[Patch v2 3/3] contrib: pick: test: refresh view

2013-08-26 Thread Mark Walters
A recent proposed patch was buggy when refreshing the view. Add a test for refresh so that this does not reoccur. --- contrib/notmuch-pick/test/emacs-pick | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/test/emacs-pick

[Patch v2 2/3] contrib: pick: add docstring for notmuch-pick-worker

2013-08-26 Thread Mark Walters
--- contrib/notmuch-pick/notmuch-pick.el |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index 5943ac2..7b13708 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++

Re: [PATCH 1/1] emacs: removed 3 duplicate functions from notmuch-show.el

2013-08-26 Thread Mark Walters
This LGTM +1 Mark On Mon, 26 Aug 2013, Tomi Ollila tomi.oll...@iki.fi wrote: notmuch-show.el and notmuch.el had 3 duplicate, identical functions: notmuch-foreach-mime-part, notmuch-count-attachments and notmuch-save-attachments. Now these functions in notmuch-show.el are replaced with

Feature proposal: OUTPUT_AUTHOR format

2013-08-26 Thread Johannes Kulick
Hi all, I have the following idea for a new feature. I'd implement it myself, but I wanted to ask the list first, whether something like this would have a chance to go into mainline (given that it is implemented cleanly, well documented etc.) or if people wouldn't like it at all. I suggest to

Re: Feature proposal: OUTPUT_AUTHOR format

2013-08-26 Thread David Bremner
Johannes Kulick johan...@mailless.org writes: An authors output format would not add redundant information to a mail, would be relatively easy to implement and I would find a people centric view - which doesn't interfere with any other concept of notmuch - very worth the couple of new lines

Re: Feature proposal: OUTPUT_AUTHOR format

2013-08-26 Thread Jani Nikula
On Aug 27, 2013 3:27 AM, David Bremner da...@tethera.net wrote: Johannes Kulick johan...@mailless.org writes: An authors output format would not add redundant information to a mail, would be relatively easy to implement and I would find a people centric view - which doesn't interfere with

[PATCH] emacs: bugfix notmuch-mua-reply when signature is present

2013-08-26 Thread Jani Nikula
On Mon, 19 Aug 2013, Tomi Ollila wrote: > On Fri, Aug 09 2013, Tomi Ollila wrote: > >> On Thu, Aug 08 2013, Geoffrey Ferrari >> wrote: >> >>> From: "Geoffrey H. Ferrari" >>> >>> When composing a reply, notmuch-mua-reply tries to be smart and cite >>> the original message by inserting it

[Patch (rebased) 0/2] contrib: pick: remove an unused variable

2013-08-26 Thread Tomi Ollila
On Sun, Aug 25 2013, Mark Walters wrote: > This is a rebased version of > id:1372712861-13515-1-git-send-email-markwalters1009 at gmail.com. > > When documenting notmuch-pick-worker I realised that > notmuch-pick-buffer-name was an essentially unused argument: it is > only used to reset itself

[PATCH] lib: fix clang build

2013-08-26 Thread Tomi Ollila
On Sun, Aug 18 2013, Jani Nikula wrote: > On Sun, 18 Aug 2013, Tomi Ollila wrote: >>> >>> -typedef struct _notmuch_string_list notmuch_string_list_t; >>> - >> >> anyway, would >> >> typedef struct visible _notmuch_string_list notmuch_string_list_t; >> >> have helped here (if yes is your

[PATCH 0/2] contrib: pick: remove sync parser

2013-08-26 Thread Mark Walters
Pick has two possible parsing methods: a synchronous one (like show) and an async one (like search). The async parser has been the default since at least the time pick first entered contrib. I don't believe there have been any reported problems with it during this time so we might as well remove

[PATCH 1/2] contrib: pick: remove sync parser

2013-08-26 Thread Mark Walters
Previously pick had the option of using an async parser like search or a sync parser like show. The async parser has always been the default and it seems fine so we can remove the sync one and the corresponding defcustom. --- contrib/notmuch-pick/notmuch-pick.el | 36

[PATCH 2/2] contrib: pick: remove sync tests

2013-08-26 Thread Mark Walters
Since the previous commit removed the sync parser we may as well remove its tests too. --- contrib/notmuch-pick/run-tests.sh |2 +- contrib/notmuch-pick/test/emacs-pick-sync | 64 - 2 files changed, 1 insertions(+), 65 deletions(-) delete mode 100755

[Patch v2 0/3] contrib: pick: remove an unused variable

2013-08-26 Thread Mark Walters
v2 of this series is at id:1377460214-4795-1-git-send-email-markwalters1009 at gmail.com v2 had a bug on refresh view (which I should have tested more). The main pick view only worked by fluke as the initial call to pick-worker was inside a let binding. This version fixes the bug, moves the call

[Patch v2 1/3] contrib: pick: remove unneeded variable notmuch-pick-buffer-name

2013-08-26 Thread Mark Walters
This variable is essentially unused: it was only used for making sure it itself got reset after a refresh of the buffer. It did this by passing an unnecessary argument to notmuch-pick-worker so remove that too. --- contrib/notmuch-pick/notmuch-pick.el | 25 + 1 files

[Patch v2 2/3] contrib: pick: add docstring for notmuch-pick-worker

2013-08-26 Thread Mark Walters
--- contrib/notmuch-pick/notmuch-pick.el |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el index 5943ac2..7b13708 100644 --- a/contrib/notmuch-pick/notmuch-pick.el +++

[Patch v2 3/3] contrib: pick: test: refresh view

2013-08-26 Thread Mark Walters
A recent proposed patch was buggy when refreshing the view. Add a test for refresh so that this does not reoccur. --- contrib/notmuch-pick/test/emacs-pick | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/contrib/notmuch-pick/test/emacs-pick

[PATCH 1/1] emacs: removed 3 duplicate functions from notmuch-show.el

2013-08-26 Thread Mark Walters
This LGTM +1 Mark On Mon, 26 Aug 2013, Tomi Ollila wrote: > notmuch-show.el and notmuch.el had 3 duplicate, identical functions: > notmuch-foreach-mime-part, notmuch-count-attachments and > notmuch-save-attachments. Now these functions in notmuch-show.el > are replaced with declare-functions

Feature proposal: OUTPUT_AUTHOR format

2013-08-26 Thread David Bremner
Johannes Kulick writes: > An authors output format would not add redundant information to a mail, would > be > relatively easy to implement and I would find a people centric view - which > doesn't interfere with any other concept of notmuch - very worth the couple of > new lines of code. > >