while not entirely unprecedented, it's unnecessary and even counter-productive to annotate directly adjacent cases with fallthrough.Jiang Xin <worldhello.net@xxxxxxxxx> writes:+++ b/pkt-line.c @@ -462,8 +462,33 @@ enum packet_read_status packet_read_with_status(int fd, char **src_buffer, } + case 2: + /* fallthrough */ + case 3:
regards