Re: [PATCH v4] log --graph: customize the graph lines with config log.graphColors

2017-01-12 Thread Duy Nguyen
Just FYI. The broken internet cables in Vietnam seem to hit my ISP really hard. It's nearly impossible to make a TCP connection. So I'm basically off the grid, hopefully not longer than two weeks. On 1/10/17, Junio C Hamano wrote: > Nguyễn Thái Ngọc Duy writes: > >> +end = string + strlen(s

Re: [PATCH v4] log --graph: customize the graph lines with config log.graphColors

2017-01-09 Thread Junio C Hamano
Nguyễn Thái Ngọc Duy writes: > + end = string + strlen(string); > + while (start < end) { > + const char *comma = strchrnul(start, ','); > + char color[COLOR_MAXLEN]; > + > + while (start < comma && isspace(*start)) > + start++; > +

[PATCH v4] log --graph: customize the graph lines with config log.graphColors

2017-01-09 Thread Nguyễn Thái Ngọc Duy
If you have a 256 colors terminal (or one with true color support), then the predefined 12 colors seem limited. On the other hand, you don't want to draw graph lines with every single color in this mode because the two colors could look extremely similar. This option allows you to hand pick the col