On Mon, Nov 12, 2018 at 6:47 AM Jeff King <peff@xxxxxxxx> wrote: > > Using strip_suffix() lets us avoid repeating ourselves. It also makes > the handling of "/" a bit less subtle (we strip one less character than > we matched in order to leave it in place, but we can just as easily > include the "/" when we add more path components). > > Signed-off-by: Jeff King <peff@xxxxxxxx> This makes sense. Thanks! (This patch caught my attention as it's a submodule thing, but now looking at the rest of the series)
