On Sun, Feb 22, 2009 at 6:48 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Felipe Contreras <felipe.contreras@xxxxxxxxx> writes: > >> + else if (actions == ACTION_GET_COLORBOOL) { >> + if (argc == 1) >> + stdout_is_tty = git_config_bool("command line", argv[0]); >> + else if (argc == 0) >> + stdout_is_tty = isatty(1); >> + return get_colorbool(argc != 0); >> + } > > I see you fixed this from the last series... Yes, argc != 1 was wrong. Now I've made sure all the tests pass. -- Felipe Contreras -- 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