comparison mercurial/color.py @ 30970:683606a829e8

color: rework conditional 'valideffect'

Not very important, but the full conditional is not that hard to follow and
having it unified make the function role a bit clearer in my opinion.

30969:ddc80d1777a6

def loadcolortable(ui, extname, colortable):
    _styles.update(colortable)

def valideffect(effect):
    'Determine if the effect is valid or not.'
    good = False
    if not _terminfo_params and effect in _effects:
        good = True
    elif (effect in _terminfo_params
          or effect[:-11] in _terminfo_params):
        good = True
    return good

30970:683606a829e8

def loadcolortable(ui, extname, colortable):
    _styles.update(colortable)

def valideffect(effect):
    'Determine if the effect is valid or not.'
    return ((not _terminfo_params and effect in _effects)
             or (effect in _terminfo_params
                 or effect[:-11] in _terminfo_params))