> On Dec 30, 2016, at 5:25 AM, Pierre-Yves David > <pierre-yves.da...@ens-lyon.org> wrote: > >> This patch adds a new 'rawrevision()' method setting the 'raw' argument to >> True >> in the revlog.revision() call that is used to differentiate changegroup >> generation and debugdata related calls to revision() from regular read >> accesses. > > I don't get that part. it seems like 'rawrevision(…)' is just 'revision(…, > raw=True)' so I do not understant why we need a new method. Can't we just > call 'revision(…, raw=True)' directly. Am I missing something? Otherwise, we > should just drop that method.
I believe it’s so that we can have extensions hook into either reading of raw revisions or non-raw revisions. In a perfect world, revision() wouldn’t even take a raw= keyword, but I’ve lost too much context this week to help without rereading the whole patch (which I don’t have time for right now.) _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel