> I worry that having three integer parameters next to each other makes it > very easy to mix them up with out getting any errors from the compiler > because the types are all compatible. Could the last two be combined > into a flags argument? A similar issues occurs in > match_stat_with_submodule() in patch 7 I'm not sure how much more I want to engineer a static helper function that is only being called in one other place. I also don't understand what you mean by combining the last two into paramters a flags argument.