[PATCH v5 0/4] indicate length of omitted body content
Peter Wang writes: > This obsoletes 1355057796-19260-1-git-send-email-markwalters1009 at gmail.com > > Peter Wang (4): > test: normalize only message filenames in show json > show: indicate charset for all omitted parts > show: indicate length, encoding of omitted body content > test: conform to content length, encoding fields Pushed, d
Re: [PATCH v5 0/4] indicate length of omitted body content
Peter Wang noval...@gmail.com writes: This obsoletes 1355057796-19260-1-git-send-email-markwalters1...@gmail.com Peter Wang (4): test: normalize only message filenames in show json show: indicate charset for all omitted parts show: indicate length, encoding of omitted body content test: conform to content length, encoding fields Pushed, d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
[PATCH v5 0/4] indicate length of omitted body content
The b versions of this look good to me. Best wishes Mark On Sat, 15 Dec 2012, David Bremner wrote: > Peter Wang writes: > >> On Sat, 15 Dec 2012 08:45:33 +, Mark Walters > gmail.com> wrote: >>> >>> This version looks good to me with one very minor comments: >>> >>> Perhaps the name format-omitted-part-meta should include sprinter (eg >>> format-omitted-part-meta-sprinter)? >> >> Seems so. I'll roll up another series if that's necessary. > > Maybe just post the one amended patch as a reply to the patch it's > amending? > > d > ___ > notmuch mailing list > notmuch at notmuchmail.org > http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH v5 0/4] indicate length of omitted body content
The b versions of this look good to me. Best wishes Mark On Sat, 15 Dec 2012, David Bremner da...@tethera.net wrote: Peter Wang noval...@gmail.com writes: On Sat, 15 Dec 2012 08:45:33 +, Mark Walters markwalters1...@gmail.com wrote: This version looks good to me with one very minor comments: Perhaps the name format-omitted-part-meta should include sprinter (eg format-omitted-part-meta-sprinter)? Seems so. I'll roll up another series if that's necessary. Maybe just post the one amended patch as a reply to the patch it's amending? d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
[PATCH v5 0/4] indicate length of omitted body content
On Sat, Dec 15 2012, David Bremner wrote: > Peter Wang writes: > >> On Sat, 15 Dec 2012 08:45:33 +, Mark Walters > gmail.com> wrote: >>> >>> This version looks good to me with one very minor comments: >>> >>> Perhaps the name format-omitted-part-meta should include sprinter (eg >>> format-omitted-part-meta-sprinter)? >> >> Seems so. I'll roll up another series if that's necessary. > > Maybe just post the one amended patch as a reply to the patch it's > amending? The 'format_omitted_part_meta' in in context of 2 patches; Are you planning to do the change sin one patch (3/4) only (or making 5/4 which changes naming) ? And... The patch series looks good to me. > > d Tomi
[PATCH v5 0/4] indicate length of omitted body content
This obsoletes 1355057796-19260-1-git-send-email-markwalters1009 at gmail.com Peter Wang (4): test: normalize only message filenames in show json show: indicate charset for all omitted parts show: indicate length, encoding of omitted body content test: conform to content length, encoding fields devel/schemata | 9 - notmuch-show.c | 36 test/crypto | 30 +- test/json| 4 +++- test/multipart | 26 ++ test/sexp| 4 +++- test/test-lib.sh | 2 +- 7 files changed, 78 insertions(+), 33 deletions(-) -- 1.7.12.1
[PATCH v5 0/4] indicate length of omitted body content
Peter Wang writes: > On Sat, 15 Dec 2012 08:45:33 +, Mark Walters gmail.com> wrote: >> >> This version looks good to me with one very minor comments: >> >> Perhaps the name format-omitted-part-meta should include sprinter (eg >> format-omitted-part-meta-sprinter)? > > Seems so. I'll roll up another series if that's necessary. Maybe just post the one amended patch as a reply to the patch it's amending? d
Re: [PATCH v5 0/4] indicate length of omitted body content
This version looks good to me with one very minor comments: Perhaps the name format-omitted-part-meta should include sprinter (eg format-omitted-part-meta-sprinter)? Best wishes Mark On Sat, 15 Dec 2012, Peter Wang noval...@gmail.com wrote: This obsoletes 1355057796-19260-1-git-send-email-markwalters1...@gmail.com Peter Wang (4): test: normalize only message filenames in show json show: indicate charset for all omitted parts show: indicate length, encoding of omitted body content test: conform to content length, encoding fields devel/schemata | 9 - notmuch-show.c | 36 test/crypto | 30 +- test/json| 4 +++- test/multipart | 26 ++ test/sexp| 4 +++- test/test-lib.sh | 2 +- 7 files changed, 78 insertions(+), 33 deletions(-) -- 1.7.12.1 ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH v5 0/4] indicate length of omitted body content
On Sat, 15 Dec 2012 08:45:33 +, Mark Walters markwalters1...@gmail.com wrote: This version looks good to me with one very minor comments: Perhaps the name format-omitted-part-meta should include sprinter (eg format-omitted-part-meta-sprinter)? Seems so. I'll roll up another series if that's necessary. Peter ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH v5 0/4] indicate length of omitted body content
Peter Wang noval...@gmail.com writes: On Sat, 15 Dec 2012 08:45:33 +, Mark Walters markwalters1...@gmail.com wrote: This version looks good to me with one very minor comments: Perhaps the name format-omitted-part-meta should include sprinter (eg format-omitted-part-meta-sprinter)? Seems so. I'll roll up another series if that's necessary. Maybe just post the one amended patch as a reply to the patch it's amending? d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
Re: [PATCH v5 0/4] indicate length of omitted body content
On Sat, Dec 15 2012, David Bremner wrote: Peter Wang noval...@gmail.com writes: On Sat, 15 Dec 2012 08:45:33 +, Mark Walters markwalters1...@gmail.com wrote: This version looks good to me with one very minor comments: Perhaps the name format-omitted-part-meta should include sprinter (eg format-omitted-part-meta-sprinter)? Seems so. I'll roll up another series if that's necessary. Maybe just post the one amended patch as a reply to the patch it's amending? The 'format_omitted_part_meta' in in context of 2 patches; Are you planning to do the change sin one patch (3/4) only (or making 5/4 which changes naming) ? And... The patch series looks good to me. d Tomi ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch
[PATCH v5 0/4] indicate length of omitted body content
This obsoletes 1355057796-19260-1-git-send-email-markwalters1...@gmail.com Peter Wang (4): test: normalize only message filenames in show json show: indicate charset for all omitted parts show: indicate length, encoding of omitted body content test: conform to content length, encoding fields devel/schemata | 9 - notmuch-show.c | 36 test/crypto | 30 +- test/json| 4 +++- test/multipart | 26 ++ test/sexp| 4 +++- test/test-lib.sh | 2 +- 7 files changed, 78 insertions(+), 33 deletions(-) -- 1.7.12.1 ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch