Hi, On Mon, 21 Jul 2008, Fredrik Tolf wrote: > I'm following my previous SSH patch up with this one, which should at > least solve the problems discussed, and probably some more. If anything, > it might be considered a bit overkill for the problem at hand. I am not assuming it is overkill, but since you do not reuse functions such as strbuf_expand() and split_cmdline(), your patch ends up pretty large. And since you use very short and undescriptive variable names, with ugly assignments inside arithmetic expressions, I will be less likely reviewing it in detail. > I assume it might have to be documented as well, if people approve of it. Catch 22. Since you have not documented what %P should be useful for, people might not approve of the patch, because they do not understand what it is supposed to do. People like me, Dscho -- 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
