[PATCH/RFC v2 5/5] send-email: refactor address list process

2015-06-08 Thread Remi Lespinet
Junio C Hamano writes > Matthieu Moy writes: > > > Remi Lespinet writes: > > > >> Simplify code by creating a function to transform list of email lists > >> (comma separated, with aliases ...) into a simple list of valid email > >> addresses. > > > > I would have found the series easier to re

Re: [PATCH/RFC v2 5/5] send-email: refactor address list process

2015-06-08 Thread Junio C Hamano
Matthieu Moy writes: > Remi Lespinet writes: > >> Simplify code by creating a function to transform list of email lists >> (comma separated, with aliases ...) into a simple list of valid email >> addresses. > > I would have found the series easier to read if this refactoring came > earlier (and

[PATCH/RFC v2 5/5] send-email: refactor address list process

2015-06-08 Thread Remi Lespinet
Matthieu Moy writes: > To me, the series is ready now, and I don't think re-rolling it would be > a good time investment. Plus, I spent time reviewing this series and > with my proposal I'd need to review a relatively different one. Ok, thanks. -- To unsubscribe from this list: send the line "un

Re: [PATCH/RFC v2 5/5] send-email: refactor address list process

2015-06-08 Thread Matthieu Moy
Remi Lespinet writes: > Matthieu Moy writes: > >> Remi Lespinet writes: >> >> > Simplify code by creating a function to transform list of email lists >> > (comma separated, with aliases ...) into a simple list of valid email >> > addresses. >> >> I would have found the series easier to read

[PATCH/RFC v2 5/5] send-email: refactor address list process

2015-06-08 Thread Remi Lespinet
Matthieu Moy writes: > Remi Lespinet writes: > > > Simplify code by creating a function to transform list of email lists > > (comma separated, with aliases ...) into a simple list of valid email > > addresses. > > I would have found the series easier to read if this refactoring came > earlier

Re: [PATCH/RFC v2 5/5] send-email: refactor address list process

2015-06-08 Thread Matthieu Moy
Remi Lespinet writes: > Simplify code by creating a function to transform list of email lists > (comma separated, with aliases ...) into a simple list of valid email > addresses. I would have found the series easier to read if this refactoring came earlier (and then PATCH 2/5 would fix the bug

[PATCH/RFC v2 5/5] send-email: refactor address list process

2015-06-06 Thread Remi Lespinet
Simplify code by creating a function to transform list of email lists (comma separated, with aliases ...) into a simple list of valid email addresses. Signed-off-by: Remi Lespinet --- I'm not sure about the name of the function... git-send-email.perl | 28 +--- 1 file