fast-import: check most prominent commands first

This is not a very important change, and one that I expect to have no performance impact whatsoever, but reading the code bothered me. The parsing of command types in cmd_main() mostly runs in order of most common to least common commands; sure, it's hard to say for sure what the most common are without some type of study, but it seems fairly clear to mark the original four ("blob", "commit", "tag", "reset") as the most prominent. Indeed, the parsing for most other commands were added to later in the list. However, when "ls" was added, it was stuck near the top of the list, with no rationale for that particular location. Move it down to later to appease my Tourette's-like internal twitching that its former location was causing. Signed-off-by: Elijah Newren <newren@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Elijah Newren committed Feb 20, 2019 at 14:58 UTC 5056bb7646cdd12d2985784f0ce4ed79550ebe63
1 file changed +2 -2
fast-import.c
+2 -2
@@ -3303,14 +3303,14 @@ int cmd_main(int argc, const char **argv)
3303 const char *v;
3304 if (!strcmp("blob", command_buf.buf))
3305 parse_new_blob();
3306 - else if (skip_prefix(command_buf.buf, "ls ", &v))
3307 - parse_ls(v, NULL);
3306 else if (skip_prefix(command_buf.buf, "commit ", &v))
3307 parse_new_commit(v);
3308 else if (skip_prefix(command_buf.buf, "tag ", &v))
3309 parse_new_tag(v);
3310 else if (skip_prefix(command_buf.buf, "reset ", &v))
3311 parse_reset_branch(v);
3312 + else if (skip_prefix(command_buf.buf, "ls ", &v))
3313 + parse_ls(v, NULL);
3314 else if (!strcmp("checkpoint", command_buf.buf))
3315 parse_checkpoint();
3316 else if (!strcmp("done", command_buf.buf))