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.