On Wed, 11 Apr 2018 07:57:56 -0400, Matt Harbison wrote:
> >> +            url = util.url(bytes(defaulturl))
> > 
> > Nit: perhaps no need to reparse the url since defaulturl is a url object.
> I don’t have the code in front of me, but I think there is an _originalpath 
> member that doesn’t get updated when any of the components do.  It shouldn’t 
> matter with the current code, but I wasn’t sure if that was ok to assume that 
> in the future.

I see. Let's leave it re-parsed then.
Mercurial-devel mailing list

Reply via email to