> If you make it so that the default value of ceil_offset is 0 (i.e. in the > absence of any GIT_CEILING_DIRS), This is what the new version of the patch does. > and at this place you did > > } while (offset > ceil_offset && cwd[--offset] != '/'); > > you wouldn't have to bend backwards with this off-by-one magic, would you? It seems like that would cause it to continue on with the outer loop, rather than aborting (which is what the current version does) when you hit the ceiling. Or maybe I'm misunderstanding something. --David -- 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