[PATCH 2/6] cli: sanitize tabs to spaces in notmuch search
Mark Walters
markwalters1009 at gmail.com
Thu Oct 17 01:07:46 PDT 2013
Hi
I have looked at the whole series and broadly it looks good. However, I
don't know this code so this is not a full review. I do have a few
comments: some of these may be plain wrong in which case my apologies!
On Wed, 16 Oct 2013, Jani Nikula <jani at nikula.org> wrote:
> This is in preparation of switching to gmime header parsing, but
> arguably converting tabs to spaces rather than question marks is the
> right thing to do anyway.
> ---
I think it would be worth saying in the commit message that this is only
for text summary output.
Also why only tabs to spaces but \n to a '?'
Best wishes
Mark
> notmuch-search.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/notmuch-search.c b/notmuch-search.c
> index d9d39ec..eab314f 100644
> --- a/notmuch-search.c
> +++ b/notmuch-search.c
> @@ -40,7 +40,9 @@ sanitize_string (const void *ctx, const char *str)
> loop = out = talloc_strdup (ctx, str);
>
> for (; *loop; loop++) {
> - if ((unsigned char)(*loop) < 32)
> + if (*loop == '\t')
> + *loop = ' ';
> + else if ((unsigned char)(*loop) < 32)
> *loop = '?';
> }
> return out;
> --
> 1.8.4.rc3
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list