Felipe Contreras wrote: > Not really. If we need to avoid the \[\], it makes sense to have a > separate function, but what I meant is that this function should be > initially on the same file, and created in a separate patch. What are you saying? 1. It makes sense to have a separate function specifically for ZSH, but it should be located in the same physical file (why?) 2. You like the original patch, but would prefer two different parts (a nit) -- 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