Daniel Barkalow <barkalow@xxxxxxxxxxxx> writes: >> > - if (!prefixcmp(url, "rsync:")) { >> > + if (remote && remote->foreign_vcs) { >> > + } else if (!prefixcmp(url, "rsync:")) { >> >> if (...) { >> ; /* empty */ >> } else ... > > I don't think I've ever tried writing an empty block for git before. It's > braces containing a semicolon and comment? (Of course, the reason I wrote > this one this way is so that the next patch could put two "+" lines in > there and have no "-" lines) Wasn't a serious "this has to be the final style" suggestion, but more about "what do you mean by this? are you going to add more code here in the later round?" question. -- 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