Thread (2 messages) 2 messages, 2 authors, 2016-06-15

[PATCH v3 7/7] send-email: suppress leading and trailing whitespaces before alias expansion

flat view
DORMANTno replies

From: Remi Lespinet <hidden>
Date: 2016-06-15 23:05:14

Matthieu Moy [off-list ref] writes:
Actually, once you have this, PATCH 6/7 becomes useless, right? (at
least, the test passes if I revert it)
It seems to me that doing this space trimming just once, inside or right
after split_at_commas would be clearer.
You're right, I put it twice because of there's occurrences of
sanitize_address which are not associated with expand_aliases, but it
seems that it's all taken care of separately in different regexp. So
there's no point to 6/7.

I agree, I'd like to put it right after split_at_commas in a separate
function "trim_list". Is it a good idea even if the function is one
line long ?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help