Jeff King <peff@xxxxxxxx> writes: > + if ((agent_feature = server_feature_value("agent", &agent_len))) { > agent_supported = 1; > + if (args.verbose && agent_len) { > + fprintf(stderr, "Server version is %.*s\n", > + agent_len, agent_feature); > + } > + } OK. The one I queued in 'pu' said "Server version not disclosed" when length was 0, but I think I like this one better. Also I like the the update to the parsing code in the previous patch. It makes the logic clearer. Thanks. -- 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