comparison mercurial/cmdutil.py @ 45036:c05ac059749f

cleanup: use any() instead of checking truthiness of temporary list

It was not immediately obvious to me, when first seeing this, why a list was
created. It needed a second look to understand that the purpose was to check
whether the condition is true for any of the parents. Using any() for that is
clearer.

45035:3862de62d5cf

    if (
        not opts.get(b'amend')
        and bheads
        and node not in bheads
        and not [
            x for x in parents if x.node() in bheads and x.branch() == branch
        ]
    ):
        repo.ui.status(_(b'created new head\n'))
        # The message is not printed for initial roots. For the other
        # changesets, it is printed in the following situations:
        #

45036:c05ac059749f

    if (
        not opts.get(b'amend')
        and bheads
        and node not in bheads
        and not any(
            x.node() in bheads and x.branch() == branch for x in parents
        )
    ):
        repo.ui.status(_(b'created new head\n'))
        # The message is not printed for initial roots. For the other
        # changesets, it is printed in the following situations:
        #