cmdstate: avoid setting a default argument to a mutable object (`{}`) If there's ever more than one cmdstate for the lifetime of the process, this can cause surprising behavior where the later cmdstates pick up options from the earlier ones. I've not seen any evidence this is actually causing any issues, but it's subtle enough that it should probably be fixed to help save significant debugging time later.

     1

# Copyright 2019 Pierre-Yves David <pierre-yves.david@octobus.net>

     2

#

     3

# This software may be used and distributed according to the terms of the

     4

# GNU General Public License version 2 or any later version.

     5

     6

def hastopicext(repo):

     7

    """True if the repo use the topic extension"""

     8

    return getattr(repo, 'hastopicext', False)