Re: [PATCH] implement search --format=sanitized_text + emacs UI to use it

Subject: Re: [PATCH] implement search --format=sanitized_text + emacs UI to use it

Date: Sat, 07 May 2011 11:14:08 +0200

To: Jameson Graef Rollins, Austin Clements, Florian Friesdorf

Cc: notmuch@notmuchmail.org, a.amann@ucc.ie

From: Pieter Praet


On Fri, 06 May 2011 18:06:47 -0700, Jameson Graef Rollins <jrollins@finestructure.net> wrote:
> On Fri, 6 May 2011 19:55:26 -0400, Austin Clements <amdragon@mit.edu> wrote:
> > Perhaps text summary output should *always* do this.  The text summary
> > format is meant half for user consumption and half for emacs
> > consumption and allowing newlines that don't indicate the end of a
> > summary line seems bad for *both* use cases.
> 
> Hi, Florian.  I think I agree with Austin here that the text output
> should probably just always be sanitized.  Can you try sending in a
> patch that just automatically sanitizes the text output?

Indeed, Andreas and I agreed [1] as well that sanitization should be
default for search, yet optional for show.

> Also, I believe that this patch is not entirely compatible with the
> current head of the release-candidate/0.6 branch.  I pushed a patch to
> that branch to fix a search output formatting bug, and in so doing I
> added a field to the search_format struct.
> 
> If you could make those changes and post a new patch that would be
> great.  Thanks.
> 
> jamie.
Non-text part: application/pgp-signature
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

Peace

-- 
Pieter


[1] id:"87pqokx7op.fsf@A7GMS.i-did-not-set--mail-host-address--so-tickle-me"

Thread: