On Mon, Nov 12, 2018 at 6:47 AM Jeff King <p...@peff.net> 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 <p...@peff.net>
This makes sense. Thanks! (This patch caught my attention as it's a submodule thing, but now looking at the rest of the series)
