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.

  rHG Mercurial


To: quark, indygreg, #hg-reviewers, ryanmce
Cc: mercurial-devel
Mercurial-devel mailing list

Reply via email to