Eric Sunshine <[email protected]> writes: >> +--atomic:: >> + Use the an atomic transaction on the server side if available. > > s/the an/an/ > ... >> + OPT_BIT(0, "atomic", &flags, N_("use a single atomic >> transaction at the serverside, if available"), > > "single atomic" sounds awfully redundant. > > "serverside" is odd. Perhaps "server side" or merely "remote" or "remote > side". > ... >> diff --git a/transport.h b/transport.h >> index 3e0091e..25fa1da 100644 >> --- a/transport.h >> +++ b/transport.h >> @@ -125,6 +125,7 @@ struct transport { >> #define TRANSPORT_PUSH_NO_HOOK 512 >> #define TRANSPORT_PUSH_FOLLOW_TAGS 1024 >> #define TRANSPORT_PUSH_CERT 2048 >> +#define TRANSPORT_ATOMIC_PUSH 4096 > > For consistency with existing names, should this be TRANSPORT_PUSH_ATOMIC?
As always, thanks for a careful reading. I missed everything you pointed out (I agree with you). -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
