Stefan Beller <sbeller@xxxxxxxxxx> writes: Overall the suggestions from you in this review is good and please consider anything I did not mention I agree with you. Thanks. >> +extern int is_submodule_initialized(const char *path); >> +extern int is_submodule_checked_out(const char *path); > > no need to put extern for function names. (no other functions in this > header are extern. so local consistency maybe? I'd also claim that > all other extern functions in headers ought to be declared without > being extern) Maybe I am old fashioned, but I'd feel better to see these with explicit "extern" in front (check the older header files like cache.h when you are in doubt what the project convention has been).