D6831: exchange: convert bookmark nodes from hex to bin ASAP

2019-09-08 Thread valentin.gatienbaron (Valentin Gatien-Baron)
valentin.gatienbaron created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D6831 AFFECTED FILES mercurial/exchange.py CHANGE DETAILS diff --git

D6829: doc: fix up confusing doc comment

2019-09-08 Thread valentin.gatienbaron (Valentin Gatien-Baron)
valentin.gatienbaron created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D6829 AFFECTED FILES mercurial/bookmarks.py mercurial/exchange.py CHANGE DETAILS diff

D6830: exchange: avoid unnecessary conversion of bookmark nodes to hex (API)

2019-09-08 Thread valentin.gatienbaron (Valentin Gatien-Baron)
valentin.gatienbaron created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D6830 AFFECTED FILES mercurial/exchange.py CHANGE DETAILS diff --git

Re: [PATCH 4 of 4] rust-cpython: mark unsafe functions as such

2019-09-08 Thread Yuya Nishihara
On Sun, 1 Sep 2019 20:46:07 +0200, Raphaël Gomès wrote: > > It wasn't trivial to fix leak_immutable() to be safe since we have to > > allow immutable operations (e.g. iter()) on the leaked reference. So > > let's mark it unsafe for now. Callers must take care of the returned > > object to

[PATCH 7 of 7] rust-cpython: leverage py_shared_iterator::from_inner() where appropriate

2019-09-08 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1567916635 -32400 # Sun Sep 08 13:23:55 2019 +0900 # Node ID 8174e0c96c14e5b5e2309a9de1a90b7c446b5aab # Parent 1fa833aa2e43f6b3055d618755f10daeefacb0ae rust-cpython: leverage py_shared_iterator::from_inner() where appropriate diff --git

[PATCH 6 of 7] rust-cpython: remove Option<_> from interface of py_shared_iterator

2019-09-08 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1567915739 -32400 # Sun Sep 08 13:08:59 2019 +0900 # Node ID 1fa833aa2e43f6b3055d618755f10daeefacb0ae # Parent d242b7375817f26ace0196768df8956033396903 rust-cpython: remove Option<_> from interface of py_shared_iterator It's the

[PATCH 5 of 7] rust-cpython: rename py_shared_iterator_impl to py_shared_iterator

2019-09-08 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1567913172 -32400 # Sun Sep 08 12:26:12 2019 +0900 # Node ID d242b7375817f26ace0196768df8956033396903 # Parent 21e85ee4785728c37823d3b3fc70b89beec4fb63 rust-cpython: rename py_shared_iterator_impl to py_shared_iterator It's a public

[PATCH 4 of 7] rust-cpython: replace dyn Iterator<..> of mapping with concrete type

2019-09-08 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1567912998 -32400 # Sun Sep 08 12:23:18 2019 +0900 # Node ID 21e85ee4785728c37823d3b3fc70b89beec4fb63 # Parent 3c707d5a55e140de8117893a142c2b17f1a7f401 rust-cpython: replace dyn Iterator<..> of mapping with concrete type See the previous

[PATCH 2 of 7] rust-dirstate: provide CopyMapIter and StateMapIter types

2019-09-08 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1567911626 -32400 # Sun Sep 08 12:00:26 2019 +0900 # Node ID 06763fe8317d686dba58464ec4ed74f8325ae825 # Parent 2c5b7c68ea1cc5141d12bd26dd6effe18fe30f44 rust-dirstate: provide CopyMapIter and StateMapIter types They will be used in the

[PATCH 1 of 7] rust-dirstate: specify concrete return type of DirsMultiset::iter()

2019-09-08 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1567911329 -32400 # Sun Sep 08 11:55:29 2019 +0900 # Node ID 2c5b7c68ea1cc5141d12bd26dd6effe18fe30f44 # Parent d9ab8eb5616cf509da1ccd3f83aa54ea3b1aa6cc rust-dirstate: specify concrete return type of DirsMultiset::iter() This allows us to

[PATCH] split: use literal syntax to build a set of one element

2019-09-08 Thread Yuya Nishihara
# HG changeset patch # User Yuya Nishihara # Date 1567726368 -32400 # Fri Sep 06 08:32:48 2019 +0900 # Node ID d9ab8eb5616cf509da1ccd3f83aa54ea3b1aa6cc # Parent 775224e26d747f9d0aee5211e34e87d728e08b29 split: use literal syntax to build a set of one element diff --git

Re: [PATCH 5 of 9] run-tests: document the `pos` variable in the matching code

2019-09-08 Thread Joerg Sonnenberger
On Sun, Sep 08, 2019 at 10:05:39AM +0200, Pierre-Yves David wrote: > > > On 9/7/19 2:59 PM, Joerg Sonnenberger wrote: > > On Sat, Sep 07, 2019 at 02:16:44PM +0200, Pierre-Yves David wrote: > > > diff --git a/tests/run-tests.py b/tests/run-tests.py > > > --- a/tests/run-tests.py > > > +++

D6808: revlog: introduce a `sidedata` method

2019-09-08 Thread marmoute (Pierre-Yves David)
marmoute added a comment. In D6808#100077 , @indygreg wrote: > I think I see where you are going with this and it seems potentially very useful. > I do wish the flag processors code could have been refactored without involving `sidedata`,

Re: [PATCH 6 of 9] run-tests: extract a `process_cmd_line` from the main function

2019-09-08 Thread Pierre-Yves David
On 9/7/19 3:03 PM, Joerg Sonnenberger wrote: On Sat, Sep 07, 2019 at 02:16:45PM +0200, Pierre-Yves David wrote: diff --git a/tests/run-tests.py b/tests/run-tests.py --- a/tests/run-tests.py +++ b/tests/run-tests.py @@ -1619,6 +1619,7 @@ class TTest(Test): if salt in out_rawline:

[PATCH 3 of 4 V2] run-tests: remove the artificial indentation

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1560530356 -3600 # Fri Jun 14 17:39:16 2019 +0100 # Node ID e01d12fd89afdda7e4d118c2e46f70f3207e1b1f # Parent 57aae8acdd093f2653922cf0a809dd861b759a59 # EXP-Topic test-match # Available At https://bitbucket.org/octobus/mercurial-devel/ #

[PATCH 4 of 4 V2] run-tests: add a dedicated 'isoptional' function

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1560531004 -3600 # Fri Jun 14 17:50:04 2019 +0100 # Node ID f70d40f8e6b66c0d885eb3d78ac2216985e3f183 # Parent e01d12fd89afdda7e4d118c2e46f70f3207e1b1f # EXP-Topic test-match # Available At https://bitbucket.org/octobus/mercurial-devel/ #

[PATCH 1 of 4 V2] run-tests: extract a `process_cmd_line` from the main function

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1567930121 -7200 # Sun Sep 08 10:08:41 2019 +0200 # Node ID 0c5eff3f149ed6a041511538ac6c04826ac0d29c # Parent 5ca351ba24788d7a74b9b2b4d35589ce87d48942 # EXP-Topic test-match # Available At https://bitbucket.org/octobus/mercurial-devel/ #

[PATCH 2 of 4 V2] run-tests: extract a `process_out_line` from the main function

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1560530224 -3600 # Fri Jun 14 17:37:04 2019 +0100 # Node ID 57aae8acdd093f2653922cf0a809dd861b759a59 # Parent 0c5eff3f149ed6a041511538ac6c04826ac0d29c # EXP-Topic test-match # Available At https://bitbucket.org/octobus/mercurial-devel/ #

Re: [PATCH 5 of 9] run-tests: document the `pos` variable in the matching code

2019-09-08 Thread Pierre-Yves David
On 9/7/19 2:59 PM, Joerg Sonnenberger wrote: On Sat, Sep 07, 2019 at 02:16:44PM +0200, Pierre-Yves David wrote: diff --git a/tests/run-tests.py b/tests/run-tests.py --- a/tests/run-tests.py +++ b/tests/run-tests.py @@ -1609,6 +1609,9 @@ class TTest(Test): if exitcode != 0:

[PATCH 5 of 5] changegroup: move message about added changes to transaction summary

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1567928573 -7200 # Sun Sep 08 09:42:53 2019 +0200 # Node ID 275367f9cadd11d65128051d920414f4b76f5de6 # Parent 426a83641e1b9a94775bd2c3db0033d141432608 # EXP-Topic check-summary # Available At https://bitbucket.org/octobus/mercurial-devel/

[PATCH 1 of 5] debugobsolete: also issue the "new obsmarkers" messsage

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1539515964 -7200 # Sun Oct 14 13:19:24 2018 +0200 # Node ID d7d52b453d9e3689b0d47685f163832ed2b20eae # Parent 69195b6f8f974ba56aec9b9bd4cd0a259a646f72 # EXP-Topic check-summary # Available At https://bitbucket.org/octobus/mercurial-devel/

[PATCH 3 of 5] narrow: rely on setting `quiet` mode instead of `pushbuffer`

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1567894280 -7200 # Sun Sep 08 00:11:20 2019 +0200 # Node ID b02758303195ae5953d2e1beacda3eec73314b20 # Parent 85d7a41d37ab5ad059320fff6857315db430fdfb # EXP-Topic check-summary # Available At https://bitbucket.org/octobus/mercurial-devel/

[PATCH 2 of 5] transaction: issue "new obsmarkers" message at the end of the transaction

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1539514742 -7200 # Sun Oct 14 12:59:02 2018 +0200 # Node ID 85d7a41d37ab5ad059320fff6857315db430fdfb # Parent d7d52b453d9e3689b0d47685f163832ed2b20eae # EXP-Topic check-summary # Available At https://bitbucket.org/octobus/mercurial-devel/

[PATCH 4 of 5] sshserver: flush stream after command dispatch

2019-09-08 Thread Pierre-Yves David
# HG changeset patch # User Pierre-Yves David # Date 1567897354 -7200 # Sun Sep 08 01:02:34 2019 +0200 # Node ID 426a83641e1b9a94775bd2c3db0033d141432608 # Parent b02758303195ae5953d2e1beacda3eec73314b20 # EXP-Topic check-summary # Available At https://bitbucket.org/octobus/mercurial-devel/