On Mon, Nov 2, 2015 at 9:41 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > > Jonathan Lebon <jonathan.lebon@xxxxxxxxx> writes: > > > As it is, the suggested command for trying out diff-highlight will just > > dump the whole git log output to the terminal. Let's pipe it through > > `less` so users aren't surprised on the first try. > > That justifies the "less" part but not your choice of "-r". > > I am assuming that you are telling "less" not to show the ANSI > "color" escape sequences using the caret notation with "-r", which > is a very natural and sensible thing to do when using `highlight`. > > But if that is the case, you don't want "-r" (raw control chars for > everything). You would want to say "-R", I think. Ahh thanks, that makes sense. I will update this for v2 tomorrow. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html