comparison mercurial/merge.py @ 28226:377f0d8ff874

merge: use any() instead of for loop when checking for dirty subrepos

I think it's both simpler and clearer to use any() than the current
for loop.

While at it, also drop the call to sorted(), since it doesn't matter
which order we iterate over subrepos.

28225:5c11702fe2a3

    copied = set(copy.values())
    copied.update(movewithdir.values())

    if '.hgsubstate' in m1:
        # check whether sub state is modified
        for s in sorted(wctx.substate):
            if wctx.sub(s).dirty():
                m1['.hgsubstate'] += '+'
                break

    # Compare manifests
    if matcher is not None:
        m1 = m1.matches(matcher)
        m2 = m2.matches(matcher)

28226:377f0d8ff874

    copied = set(copy.values())
    copied.update(movewithdir.values())

    if '.hgsubstate' in m1:
        # check whether sub state is modified
        if any(wctx.sub(s).dirty() for s in wctx.substate):
            m1['.hgsubstate'] += '+'

    # Compare manifests
    if matcher is not None:
        m1 = m1.matches(matcher)
        m2 = m2.matches(matcher)