Michael Haggerty <mhagger@xxxxxxxxxxxx> writes: > Sorry I didn't respond to this earlier. > > The public API convention (which predates my changes) is that "char > *submodule" arguments either point at the relative path to the submodule > or are NULL to denote the main module. But since these are stored > internally in a name[FLEX_ARRAY] field, I have been using "" internally > to denote the main module. I believe that everything is done correctly, > but I admit that the use of different conventions internally and > externally is a potential source of programming errors. Yes, it would have been better if the original also used "". After all, that would make it more consistent---"sub/" means the repository goverend by "sub/.git", and "" would mean the repository governed by ".git". Is it hard to change to do so now, given that we won't be rushing this for the upcoming release and we have plenty of time? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html