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:25:43 +0200

To: Jameson Graef Rollins, Austin Clements, Florian Friesdorf

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

From: Pieter Praet


On Sat, 07 May 2011 11:14:08 +0200, Pieter Praet <pieter@praet.org> wrote:
> 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"

... and apparently that never reached the list since his reply was
addressed at me personally, so I've forwarded it: 87mxiykgu3.fsf@praet.org


Peace

-- 
Pieter

Thread: