D6808: revlog: introduce a `sidedata` method

2019-09-27 Thread marmoute (Pierre-Yves David)
Closed by commit rHG0d1272783f24: revlog: introduce a `sidedata` method (authored by marmoute). This revision was automatically updated to reflect the committed changes. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6808?vs=16474=16628 CHANGES SINCE

D6808: revlog: introduce a `sidedata` method

2019-09-09 Thread marmoute (Pierre-Yves David)
marmoute added a comment. The next series (adding actual storage and reading is available here: https://bitbucket.org/octobus/mercurial-devel/commits/?search=777a564525e8%3A%3Ae9075930b1b7) REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6808/new/

D6808: revlog: introduce a `sidedata` method

2019-09-09 Thread marmoute (Pierre-Yves David)
marmoute updated this revision to Diff 16474. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D6808?vs=16413=16474 CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6808/new/ REVISION DETAIL https://phab.mercurial-scm.org/D6808 AFFECTED FILES

D6808: revlog: introduce a `sidedata` method

2019-09-09 Thread durin42 (Augie Fackler)
durin42 added a comment. FWIW I'm +0.5 on this because it seems like a reasonable abstraction and could serve a variety of purposes. REPOSITORY rHG Mercurial CHANGES SINCE LAST ACTION https://phab.mercurial-scm.org/D6808/new/ REVISION DETAIL https://phab.mercurial-scm.org/D6808 To:

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`,

D6808: revlog: introduce a `sidedata` method

2019-09-07 Thread indygreg (Gregory Szorc)
indygreg added a comment. 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`, as I would have taken the patches later in this series to remove the temporary `mixin`

D6808: revlog: introduce a `sidedata` method

2019-09-07 Thread marmoute (Pierre-Yves David)
marmoute created this revision. marmoute added reviewers: yuja, durin42. Herald added a reviewer: indygreg. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY The method give access to extra information related to the revision. Such data will