[PATCH v5 2/3] color.c: trim leading spaces in color_parse_mem()

2017-01-19 Thread Nguyễn Thái Ngọc Duy
Normally color_parse_mem() is called from config parser which trims the leading spaces already. The new caller in the next patch won't. Let's be tidy and trim leading spaces too (we already trim trailing spaces before comma). Signed-off-by: Nguyễn Thái Ngọc Duy --- color.c | 7 ++- 1 file ch

Re: [PATCH v5 2/3] color.c: trim leading spaces in color_parse_mem()

2017-01-19 Thread Jeff King
On Thu, Jan 19, 2017 at 06:41:22PM +0700, Nguyễn Thái Ngọc Duy wrote: > Normally color_parse_mem() is called from config parser which trims the > leading spaces already. The new caller in the next patch won't. Let's be > tidy and trim leading spaces too (we already trim trailing spaces before > co

Re: [PATCH v5 2/3] color.c: trim leading spaces in color_parse_mem()

2017-01-19 Thread Junio C Hamano
Jeff King writes: > On Thu, Jan 19, 2017 at 06:41:22PM +0700, Nguyễn Thái Ngọc Duy wrote: > >> Normally color_parse_mem() is called from config parser which trims the >> leading spaces already. The new caller in the next patch won't. Let's be >> tidy and trim leading spaces too (we already trim t