On 09/27, Junio C Hamano wrote: > Brandon Williams <bmwill@xxxxxxxxxx> writes: > > > +/* Returns 1 if packet_buffer is a protocol version pkt-line, 0 otherwise. */ > > +static int process_protocol_version(void) > > +{ > > + switch (determine_protocol_version_client(packet_buffer)) { > > + case protocol_v1: > > + return 1; > > + case protocol_v0: > > + return 0; > > + default: > > + die("server is speaking an unknown protocol"); > > + } > > +} > > checkpatch.pl yells at me: > > ERROR: switch and case should be at the same indent > > and we would probably want to teach "make style" the same, if we > already don't. 'make style' actually already understands this, I just forgot it run it on this change :) -- Brandon Williams