Jon Seymour <jon.seymour@xxxxxxxxx> writes: > +bisect_head() > +{ > + if test -f "$GIT_DIR/BISECT_HEAD"; then > + echo BISECT_HEAD; > + else if test -f ... then echo BISECT_HEAD else > + --no-checkout) > + mode=--no-checkout; You seem to be very fond of extraneous semicolons. Also I very much prefer to write "then" on its own line, indented at the same level as "if". Consistently doing so will make your if statement a lot easier to read when some of them need to have multi-line conditionals. -- 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