[PATCH v2] lib: Fix name reordering to handle commas without spaces
Tomi Ollila
tomi.ollila at iki.fi
Mon Feb 25 22:47:56 PST 2013
On Tue, Feb 26 2013, Adam Wolfe Gordon <awg+notmuch at xvx.ca> wrote:
> Notmuch automatically re-orders names of the format "Last, First" to
> "First Last" when the associated email address is
> First.Last at example.com. But, if a name is of the format "Last,First"
> then notmuch will format the name as "irst Last". Handle any number of
> spaces after the comma, including none.
> ---
>
> Good idea, Tomi. This version handles any number of spaces.
LGTM.
Tomi
>
> lib/thread.cc | 12 ++++++++++--
> 1 file changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/lib/thread.cc b/lib/thread.cc
> index e976d64..ce6b0ef 100644
> --- a/lib/thread.cc
> +++ b/lib/thread.cc
> @@ -186,8 +186,16 @@ _thread_cleanup_author (notmuch_thread_t *thread,
> if (comma && strlen(comma) > 1) {
> /* let's assemble what we think is the correct name */
> lname = comma - author;
> - fname = strlen(author) - lname - 2;
> - strncpy(clean_author, comma + 2, fname);
> +
> + /* Skip all the spaces after the comma */
> + fname = strlen(author) - lname - 1;
> + comma += 1;
> + while (*comma == ' ') {
> + fname -= 1;
> + comma += 1;
> + }
> + strncpy(clean_author, comma, fname);
> +
> *(clean_author+fname) = ' ';
> strncpy(clean_author + fname + 1, author, lname);
> *(clean_author+fname+1+lname) = '\0';
> --
> 1.7.9.5
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
More information about the notmuch
mailing list