indygreg accepted this revision.
indygreg added a comment.
This revision is now accepted and ready to land.
This change is a bit difficult to reason about. But I think it is fine.
Essentially, we're ensuring that the delta stored in the revlog is applied on
top of the fulltext after flag transformation rather than before. In the common
case where we have no flag processors, this doesn't matter. In the case of LFS,
it ensures LFS content resolution is performed first so we can delta against
On the addrevision side, we're now taking the size of the flag resolved
content rather than the raw content. In the common case of no flag processors,
the new behavior is identical. In the case of LFS, we compute the size of the
LFS resolved data rather than the pointer record.
To: quark, indygreg, #hg-reviewers, ryanmce
Mercurial-devel mailing list