comparison hgext/phabricator.py @ 44608:7b9aaec17126

phabricator: eliminate a couple of duplicate filectx lookups

Differential Revision: https://phab.mercurial-scm.org/D8305

44607:a7b7a67b79c1

    """add removed files to the phabdiff. Shouldn't include moves"""
    for fname in removed:
        pchange = phabchange(
            currentPath=fname, oldPath=fname, type=DiffChangeType.DELETE
        )
        pchange.addoldmode(gitmode[ctx.p1()[fname].flags()])
        oldfctx = ctx.p1()[fname]
        if not (oldfctx.isbinary() or notutf8(oldfctx)):
            maketext(pchange, ctx, fname)

        pdiff.addchange(pchange)


def addmodified(pdiff, ctx, modified):
    """add modified files to the phabdiff"""
    for fname in modified:
        fctx = ctx[fname]
        oldfctx = fctx.p1()
        pchange = phabchange(currentPath=fname, oldPath=fname)
        filemode = gitmode[ctx[fname].flags()]
        originalmode = gitmode[ctx.p1()[fname].flags()]
        if filemode != originalmode:
            pchange.addoldmode(originalmode)
            pchange.addnewmode(filemode)

        if (

            or notutf8(fctx)
            or oldfctx.isbinary()
            or notutf8(oldfctx)
        ):
            makebinary(pchange, fctx)
            addoldbinary(pchange, fctx.p1(), fctx)
        else:
            maketext(pchange, ctx, fname)

        pdiff.addchange(pchange)

    for fname in added:
        fctx = ctx[fname]
        oldfctx = None
        pchange = phabchange(currentPath=fname)

        filemode = gitmode[ctx[fname].flags()]
        renamed = fctx.renamed()

        if renamed:
            originalfname = renamed[0]
            oldfctx = ctx.p1()[originalfname]

44608:7b9aaec17126

    """add removed files to the phabdiff. Shouldn't include moves"""
    for fname in removed:
        pchange = phabchange(
            currentPath=fname, oldPath=fname, type=DiffChangeType.DELETE
        )
        oldfctx = ctx.p1()[fname]
        pchange.addoldmode(gitmode[oldfctx.flags()])
        if not (oldfctx.isbinary() or notutf8(oldfctx)):
            maketext(pchange, ctx, fname)

        pdiff.addchange(pchange)


def addmodified(pdiff, ctx, modified):
    """add modified files to the phabdiff"""
    for fname in modified:
        fctx = ctx[fname]
        oldfctx = ctx.p1()[fname]
        pchange = phabchange(currentPath=fname, oldPath=fname)
        filemode = gitmode[fctx.flags()]
        originalmode = gitmode[oldfctx.flags()]
        if filemode != originalmode:
            pchange.addoldmode(originalmode)
            pchange.addnewmode(filemode)

        if (

            or notutf8(fctx)
            or oldfctx.isbinary()
            or notutf8(oldfctx)
        ):
            makebinary(pchange, fctx)
            addoldbinary(pchange, oldfctx, fctx)
        else:
            maketext(pchange, ctx, fname)

        pdiff.addchange(pchange)

    for fname in added:
        fctx = ctx[fname]
        oldfctx = None
        pchange = phabchange(currentPath=fname)

        filemode = gitmode[fctx.flags()]
        renamed = fctx.renamed()

        if renamed:
            originalfname = renamed[0]
            oldfctx = ctx.p1()[originalfname]