D1253: dirstate: avoid reading the map when possible (issue5713) (issue5717)

2017-10-26 Thread mharbison72 (Matt Harbison)
mharbison72 added a comment. Try that again without the reformatting diff --git a/tests/test-static-http.t b/tests/test-static-http.t --- a/tests/test-static-http.t +++ b/tests/test-static-http.t @@ -221,34 +221,31 @@ $ cat server.log | sed -n -e 's|.*GET \(/[^

D1253: dirstate: avoid reading the map when possible (issue5713) (issue5717)

2017-10-26 Thread mharbison72 (Matt Harbison)
mharbison72 added a comment. This makes Windows mostly happy again, but now I'm getting the following diff. Any ideas? diff --git a/tests/test-static-http.t b/tests/test-static-http.t - a/tests/test-static-http.t +++ b/tests/test-static-http.t @@ -221,34 +221,31 @@ $

Re: D1167: makefile: add target to apply clang-format in-place

2017-10-26 Thread Matt Harbison
On Wed, 25 Oct 2017 23:42:04 -0400, av6 (Anton Shestakov) wrote: av6 added a comment. I'm a bit late to the party, but wanted to remind people that targets that are not actual files (in this case, a file named "format-c") should also go into .PHONY

D1253: dirstate: avoid reading the map when possible (issue5713) (issue5717)

2017-10-26 Thread durham (Durham Goode)
durham created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Before the recent refactor, we would not load the entire map until it was accessed. As part of the refactor, that got lost and even just trying to load the

D1252: dirstate: move clear onto dirstatemap class

2017-10-26 Thread durham (Durham Goode)
durham created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY A future diff will move the lazy loading aspect of dirstate to the dirstatemap class. This means it requires a slightly different strategy of clearing than just

[PATCH] filemerge: pass a default value to _toolstr

2017-10-26 Thread Kostia Balytskyi
# HG changeset patch # User Kostia Balytskyi # Date 1509041226 25200 # Thu Oct 26 11:07:06 2017 -0700 # Branch stable # Node ID 81db944aae69a96175fd0d917834e67534a92735 # Parent 9e9ff559440e4733fda49ddeb1b9ce5e7c9660af filemerge: pass a default value to _toolstr After a

Re: [PATCH stable] makefile: put format-c into .PHONY targets

2017-10-26 Thread Yuya Nishihara
On Thu, 26 Oct 2017 12:00:48 +0800, Anton Shestakov wrote: > # HG changeset patch > # User Anton Shestakov > # Date 1508990335 -28800 > # Thu Oct 26 11:58:55 2017 +0800 > # Branch stable > # Node ID 8ed12ff6d0edcb3f28d2d88ddb63397ea9539ba5 > # Parent

Re: Deploying registrar configs in a BC safe way

2017-10-26 Thread Boris Feld
On Wed, 2017-10-25 at 21:33 -0400, Augie Fackler wrote: > A wrinkle I noticed in hgsubversion tonight: I've got a fair number > of config items whose default is `None`, but the config registrar > treats a default of None as no default and requires specifying a > default explicitly. That's weird,