Duy Nguyen <pclouds@xxxxxxxxx> writes: >> This is to handle a corrupt input, e.g. "%C(auto)%Cbleu%H" where >> (perhaps deprecated) "%Cblue" is misspelled, and parse_color() >> returns 0 without consuming any byte. >> >> Does it make sense not to turn auto off in such a case? > > We don't have any mechanism to report invalid %C. Instead we let them > through as literals. If they are literals, they should not have any > side effects. So I think it makes sense not to turn off things. Oh, you are right. Thanks for a dose of sanity. -- 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