Patchwork D8219: phabricator: rename a variable to clarify that it is the parent filecontext

login
register
mail settings
Submitter phabricator
Date March 4, 2020, 4:56 p.m.
Message ID <differential-rev-PHID-DREV-ltbdxhpgyvs67vd7ubas-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/45472/
State Superseded
Headers show

Comments

phabricator - March 4, 2020, 4:56 p.m.
mharbison72 created this revision.
Herald added subscribers: mercurial-devel, Kwan.
Herald added a reviewer: hg-reviewers.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D8219

AFFECTED FILES
  hgext/phabricator.py

CHANGE DETAILS




To: mharbison72, #hg-reviewers
Cc: Kwan, mercurial-devel

Patch

diff --git a/hgext/phabricator.py b/hgext/phabricator.py
--- a/hgext/phabricator.py
+++ b/hgext/phabricator.py
@@ -814,8 +814,8 @@ 
             currentPath=fname, oldPath=fname, type=DiffChangeType.DELETE
         )
         pchange.addoldmode(gitmode[ctx.p1()[fname].flags()])
-        fctx = ctx.p1()[fname]
-        if not (fctx.isbinary() or notutf8(fctx)):
+        oldfctx = ctx.p1()[fname]
+        if not (oldfctx.isbinary() or notutf8(oldfctx)):
             maketext(pchange, ctx, fname)
 
         pdiff.addchange(pchange)