[PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread David Bremner
Tomi Ollila writes: > > id:"1337152910-22371-4-git-send-email-tomi.ollila at iki.fi" > id:"1337152910-22371-5-git-send-email-tomi.ollila at iki.fi" > id:"1337152910-22371-6-git-send-email-tomi.ollila at iki.fi" > id:"1337152910-22371-7-git-send-email-tomi.ollila at iki.fi" I pushed those to

[PATCH 6/6] cli: lazily create the crypto gpg context only when needed

2012-05-16 Thread Jameson Graef Rollins
Move the creation of the crypto ctx into mime-node.c and create it only when needed. This removes code duplication from notmuch-show and notmuch-reply, and should speed up these functions considerably if the crypto flags are provided but the messages don't have any cryptographic parts. ---

[PATCH 5/6] cli: new crypto verify flag to handle verification

2012-05-16 Thread Jameson Graef Rollins
Use this flag rather than depend on the existence of an initialized gpgctx, to determine whether we should verify a multipart/signed. We will be moving to create the ctx lazily, so we don't want to depend on it being previously initialized if it's not needed. --- mime-node.c |5 ++---

[PATCH 4/6] cli: intialize crypto structure in show and reply

2012-05-16 Thread Jameson Graef Rollins
This makes sure it has proper initialization values when it's created. --- notmuch-reply.c |5 - notmuch-show.c | 10 +- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/notmuch-reply.c b/notmuch-reply.c index 6662adb..3c967a0 100644 --- a/notmuch-reply.c +++

[PATCH 3/6] cli: modify mime_node_open to take crypto struct as argument

2012-05-16 Thread Jameson Graef Rollins
Again, for interface simplification and getting rid of more #ifdefs. --- mime-node.c | 10 ++ notmuch-client.h | 14 +- notmuch-reply.c |6 ++ notmuch-show.c |3 +-- 4 files changed, 10 insertions(+), 23 deletions(-) diff --git a/mime-node.c

[PATCH 2/6] cli: modify mime_node_context to use the new notmuch_crypto_t

2012-05-16 Thread Jameson Graef Rollins
This simplifies some more interfaces and gets rid of another #ifdef. --- mime-node.c | 23 +-- 1 file changed, 9 insertions(+), 14 deletions(-) diff --git a/mime-node.c b/mime-node.c index a95bdab..79a3654 100644 --- a/mime-node.c +++ b/mime-node.c @@ -33,12 +33,7 @@

[PATCH 1/6] cli: new crypto structure to store crypto contexts and parameters

2012-05-16 Thread Jameson Graef Rollins
The main point here is to keep track of the crypto stuff together in one place. In notmuch-show the crypto struct is a sub structure of the parameters struct. In notmuch-reply, which had been using a notmuch_show_params_t to store the crypto parameters, we can now just use the general crypto

[PATCH 0/6] cli: improve handling of crypto parameters contexts

2012-05-16 Thread Jameson Graef Rollins
I wanted to see if I could hack in S/MIME support, but ultimately failed (the interface is too stupid for me to deal with at the moment). However, on the way there I did manage to make some tangible improvements to how crypto is handled in show and reply. Most importantly I've moved the

emacs complains about encoding?

2012-05-16 Thread Tomi Ollila
On Wed, May 16 2012, Jani Nikula wrote: > Tomi Ollila writes: >> IT h\344ppens. >> >> Attempting to reply to this email should expose the problem. >> >> \304\344li\366 \344l\344 ly\366, \366\366li\344 l\344ikkyy. (\nnn text above hand-(query)-replaced) > > The problem bisected to

[PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Tomi Ollila
Sorry about the spam -- sent too many patches... PEBKAC. The only relevant are: id:"1337152910-22371-4-git-send-email-tomi.ollila at iki.fi" id:"1337152910-22371-5-git-send-email-tomi.ollila at iki.fi" id:"1337152910-22371-6-git-send-email-tomi.ollila at iki.fi"

[PATCH 4/4] NEWS: Capitalized go bindings changes title

2012-05-16 Thread Tomi Ollila
Align 'Go bindings changes' title capitalization to rest of the file --- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index ad4859d..668281c 100644 --- a/NEWS +++ b/NEWS @@ -103,7 +103,7 @@ The function `notmuch_database_close` has been split into

[PATCH 3/4] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Tomi Ollila
NEWS entries in section 0.13 is brought consistent with rest of the NEWS file. --- NEWS | 34 +- 1 files changed, 17 insertions(+), 17 deletions(-) diff --git a/NEWS b/NEWS index 970aa40..ad4859d 100644 --- a/NEWS +++ b/NEWS @@ -6,12 +6,12 @@ Command-Line

[PATCH 2/4] NEWS: Changed 0.13 release date in NEWS file to 2012-05-15

2012-05-16 Thread Tomi Ollila
--- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index 9a690cb..970aa40 100644 --- a/NEWS +++ b/NEWS @@ -1,4 +1,4 @@ -Notmuch 0.13 (2012-xx-xx) +Notmuch 0.13 (2012-05-15) = Command-Line Interface -- 1.7.1

[PATCH 1/4] NEWS: Dropped old 'Reply to sender' section

2012-05-16 Thread Tomi Ollila
'Reply to sender' section was 0.12 news which was accidentally duplicated in 0.13 news --- NEWS |7 --- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/NEWS b/NEWS index 69a7203..9a690cb 100644 --- a/NEWS +++ b/NEWS @@ -4,13 +4,6 @@ Notmuch 0.13 (2012-xx-xx) Command-Line

[PATCH 3/3] Capitalized go bindings changes title

2012-05-16 Thread Tomi Ollila
Align 'Go bindings changes' title capitalization to rest of the file --- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index 8a73e08..4978261 100644 --- a/NEWS +++ b/NEWS @@ -110,7 +110,7 @@ The function `notmuch_database_close` has been split into

[PATCH 2/3] Changed 0.13 release date in NEWS file to 2012-05-15

2012-05-16 Thread Tomi Ollila
--- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index d841a4c..8a73e08 100644 --- a/NEWS +++ b/NEWS @@ -1,4 +1,4 @@ -Notmuch 0.13 (2012-xx-xx) +Notmuch 0.13 (2012-05-15) = Command-Line Interface -- 1.7.1

[PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Tomi Ollila
NEWS entries in section 0.13 is brought consistent with rest of the NEWS file. --- NEWS | 38 +++--- 1 files changed, 19 insertions(+), 19 deletions(-) diff --git a/NEWS b/NEWS index 69a7203..d841a4c 100644 --- a/NEWS +++ b/NEWS @@ -6,19 +6,19 @@ Command-Line

[PATCH 3/3] Capitalized go bindings changes title

2012-05-16 Thread Tomi Ollila
Align 'Go bindings changes' title capitalization to rest of the file --- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index 8a73e08..4978261 100644 --- a/NEWS +++ b/NEWS @@ -110,7 +110,7 @@ The function `notmuch_database_close` has been split into

[PATCH 2/3] Changed 0.13 release date in NEWS file to 2012-05-15

2012-05-16 Thread Tomi Ollila
--- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index d841a4c..8a73e08 100644 --- a/NEWS +++ b/NEWS @@ -1,4 +1,4 @@ -Notmuch 0.13 (2012-xx-xx) +Notmuch 0.13 (2012-05-15) = Command-Line Interface -- 1.7.1

[PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Tomi Ollila
NEWS entries in section 0.13 is brought consistent with rest of the NEWS file. --- NEWS | 38 +++--- 1 files changed, 19 insertions(+), 19 deletions(-) diff --git a/NEWS b/NEWS index 69a7203..d841a4c 100644 --- a/NEWS +++ b/NEWS @@ -6,19 +6,19 @@ Command-Line

emacs complains about encoding?

2012-05-16 Thread Jani Nikula
Tomi Ollila writes: > IT h?ppens. > > Attempting to reply to this email should expose the problem. > > ??li? ?l? ly?, ??li? l?ikkyy. The problem bisected to f6c170fabca8f39e74705e3813504137811bf162 ("emacs: Correctly quote non-text/plain parts in reply"). The commit reverts cleanly, replying

emacs complains about encoding?

2012-05-16 Thread Tomi Ollila
On Tue, May 15 2012, Tomi Ollila wrote: > IT h?ppens. > > Attempting to reply to this email should expose the problem. > > ??li? ?l? ly?, ??li? l?ikkyy. This email: id:"20120515194455.B7AD5100646 at guru.guru-group.fi" was supposed to be reply to email id:"878vgzrvik.fsf at beesknees.cern.ch"

Re: emacs complains about encoding?

2012-05-16 Thread Jani Nikula
Tomi Ollila tomi.oll...@iki.fi writes: IT häppens. Attempting to reply to this email should expose the problem. Ääliö älä lyö, ööliä läikkyy. The problem bisected to f6c170fabca8f39e74705e3813504137811bf162 (emacs: Correctly quote non-text/plain parts in reply). The commit reverts cleanly,

Re: [PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Jani Nikula
Tomi Ollila tomi.oll...@iki.fi writes: NEWS entries in section 0.13 is brought consistent with rest of the NEWS file. --- NEWS | 38 +++--- 1 files changed, 19 insertions(+), 19 deletions(-) diff --git a/NEWS b/NEWS index 69a7203..d841a4c 100644 ---

[PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Tomi Ollila
NEWS entries in section 0.13 is brought consistent with rest of the NEWS file. --- NEWS | 38 +++--- 1 files changed, 19 insertions(+), 19 deletions(-) diff --git a/NEWS b/NEWS index 69a7203..d841a4c 100644 --- a/NEWS +++ b/NEWS @@ -6,19 +6,19 @@ Command-Line

[PATCH 2/3] Changed 0.13 release date in NEWS file to 2012-05-15

2012-05-16 Thread Tomi Ollila
--- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index d841a4c..8a73e08 100644 --- a/NEWS +++ b/NEWS @@ -1,4 +1,4 @@ -Notmuch 0.13 (2012-xx-xx) +Notmuch 0.13 (2012-05-15) = Command-Line Interface -- 1.7.1

[PATCH 3/3] Capitalized go bindings changes title

2012-05-16 Thread Tomi Ollila
Align 'Go bindings changes' title capitalization to rest of the file --- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index 8a73e08..4978261 100644 --- a/NEWS +++ b/NEWS @@ -110,7 +110,7 @@ The function `notmuch_database_close` has been split into

[PATCH 1/4] NEWS: Dropped old 'Reply to sender' section

2012-05-16 Thread Tomi Ollila
'Reply to sender' section was 0.12 news which was accidentally duplicated in 0.13 news --- NEWS |7 --- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/NEWS b/NEWS index 69a7203..9a690cb 100644 --- a/NEWS +++ b/NEWS @@ -4,13 +4,6 @@ Notmuch 0.13 (2012-xx-xx) Command-Line

[PATCH 2/4] NEWS: Changed 0.13 release date in NEWS file to 2012-05-15

2012-05-16 Thread Tomi Ollila
--- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index 9a690cb..970aa40 100644 --- a/NEWS +++ b/NEWS @@ -1,4 +1,4 @@ -Notmuch 0.13 (2012-xx-xx) +Notmuch 0.13 (2012-05-15) = Command-Line Interface -- 1.7.1

[PATCH 4/4] NEWS: Capitalized go bindings changes title

2012-05-16 Thread Tomi Ollila
Align 'Go bindings changes' title capitalization to rest of the file --- NEWS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/NEWS b/NEWS index ad4859d..668281c 100644 --- a/NEWS +++ b/NEWS @@ -103,7 +103,7 @@ The function `notmuch_database_close` has been split into

[PATCH 3/4] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Tomi Ollila
NEWS entries in section 0.13 is brought consistent with rest of the NEWS file. --- NEWS | 34 +- 1 files changed, 17 insertions(+), 17 deletions(-) diff --git a/NEWS b/NEWS index 970aa40..ad4859d 100644 --- a/NEWS +++ b/NEWS @@ -6,12 +6,12 @@ Command-Line

Re: [PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread Tomi Ollila
Sorry about the spam -- sent too many patches... PEBKAC. The only relevant are: id:1337152910-22371-4-git-send-email-tomi.oll...@iki.fi id:1337152910-22371-5-git-send-email-tomi.oll...@iki.fi id:1337152910-22371-6-git-send-email-tomi.oll...@iki.fi

Re: emacs complains about encoding?

2012-05-16 Thread Tomi Ollila
On Wed, May 16 2012, Jani Nikula wrote: Tomi Ollila tomi.oll...@iki.fi writes: IT h\344ppens. Attempting to reply to this email should expose the problem. \304\344li\366 \344l\344 ly\366, \366\366li\344 l\344ikkyy. (\nnn text above hand-(query)-replaced) The problem bisected to

[PATCH 3/6] cli: modify mime_node_open to take crypto struct as argument

2012-05-16 Thread Jameson Graef Rollins
Again, for interface simplification and getting rid of more #ifdefs. --- mime-node.c | 10 ++ notmuch-client.h | 14 +- notmuch-reply.c |6 ++ notmuch-show.c |3 +-- 4 files changed, 10 insertions(+), 23 deletions(-) diff --git a/mime-node.c

[PATCH 0/6] cli: improve handling of crypto parameters contexts

2012-05-16 Thread Jameson Graef Rollins
I wanted to see if I could hack in S/MIME support, but ultimately failed (the interface is too stupid for me to deal with at the moment). However, on the way there I did manage to make some tangible improvements to how crypto is handled in show and reply. Most importantly I've moved the

[PATCH 2/6] cli: modify mime_node_context to use the new notmuch_crypto_t

2012-05-16 Thread Jameson Graef Rollins
This simplifies some more interfaces and gets rid of another #ifdef. --- mime-node.c | 23 +-- 1 file changed, 9 insertions(+), 14 deletions(-) diff --git a/mime-node.c b/mime-node.c index a95bdab..79a3654 100644 --- a/mime-node.c +++ b/mime-node.c @@ -33,12 +33,7 @@

[PATCH 6/6] cli: lazily create the crypto gpg context only when needed

2012-05-16 Thread Jameson Graef Rollins
Move the creation of the crypto ctx into mime-node.c and create it only when needed. This removes code duplication from notmuch-show and notmuch-reply, and should speed up these functions considerably if the crypto flags are provided but the messages don't have any cryptographic parts. ---

[PATCH 4/6] cli: intialize crypto structure in show and reply

2012-05-16 Thread Jameson Graef Rollins
This makes sure it has proper initialization values when it's created. --- notmuch-reply.c |5 - notmuch-show.c | 10 +- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/notmuch-reply.c b/notmuch-reply.c index 6662adb..3c967a0 100644 --- a/notmuch-reply.c +++

[PATCH 5/6] cli: new crypto verify flag to handle verification

2012-05-16 Thread Jameson Graef Rollins
Use this flag rather than depend on the existence of an initialized gpgctx, to determine whether we should verify a multipart/signed. We will be moving to create the ctx lazily, so we don't want to depend on it being previously initialized if it's not needed. --- mime-node.c |5 ++---

Re: [PATCH 1/3] NEWS: Insert markdown formatting commands in 0.13 section text

2012-05-16 Thread David Bremner
Tomi Ollila tomi.oll...@iki.fi writes: id:1337152910-22371-4-git-send-email-tomi.oll...@iki.fi id:1337152910-22371-5-git-send-email-tomi.oll...@iki.fi id:1337152910-22371-6-git-send-email-tomi.oll...@iki.fi id:1337152910-22371-7-git-send-email-tomi.oll...@iki.fi I pushed those to release.