diff options
author | Benjamin Peterson <benjamin@python.org> | 2010-03-02 23:02:02 (GMT) |
---|---|---|
committer | Benjamin Peterson <benjamin@python.org> | 2010-03-02 23:02:02 (GMT) |
commit | 0e717addd8a03ed10231a578fce894c07ddb157e (patch) | |
tree | 58c5c39ae883554b0b6b7f61156770861872ac3c /Lib | |
parent | bd56722a274287b4ce6212569aff546b572923ab (diff) | |
download | cpython-0e717addd8a03ed10231a578fce894c07ddb157e.zip cpython-0e717addd8a03ed10231a578fce894c07ddb157e.tar.gz cpython-0e717addd8a03ed10231a578fce894c07ddb157e.tar.bz2 |
remove cross-version compatibility code
Diffstat (limited to 'Lib')
-rw-r--r-- | Lib/argparse.py | 35 | ||||
-rw-r--r-- | Lib/test/test_argparse.py | 24 |
2 files changed, 9 insertions, 50 deletions
diff --git a/Lib/argparse.py b/Lib/argparse.py index 34ddb61..5148d64 100644 --- a/Lib/argparse.py +++ b/Lib/argparse.py @@ -95,27 +95,6 @@ import textwrap as _textwrap from gettext import gettext as _ -try: - _set = set -except NameError: - from sets import Set as _set - -try: - _basestring = basestring -except NameError: - _basestring = str - -try: - _sorted = sorted -except NameError: - - def _sorted(iterable, reverse=False): - result = list(iterable) - result.sort() - if reverse: - result.reverse() - return result - def _callable(obj): return hasattr(obj, '__call__') or hasattr(obj, '__bases__') @@ -152,7 +131,7 @@ class _AttributeHolder(object): return '%s(%s)' % (type_name, ', '.join(arg_strings)) def _get_kwargs(self): - return _sorted(self.__dict__.items()) + return sorted(self.__dict__.items()) def _get_args(self): return [] @@ -405,7 +384,7 @@ class HelpFormatter(object): def _format_actions_usage(self, actions, groups): # find group indices and identify actions in groups - group_actions = _set() + group_actions = set() inserts = {} for group in groups: try: @@ -475,7 +454,7 @@ class HelpFormatter(object): parts.append(part) # insert things at the necessary indices - for i in _sorted(inserts, reverse=True): + for i in sorted(inserts, reverse=True): parts[i:i] = [inserts[i]] # join all the action items with spaces @@ -1705,7 +1684,7 @@ class ArgumentParser(_AttributeHolder, _ActionsContainer): if not hasattr(namespace, action.dest): if action.default is not SUPPRESS: default = action.default - if isinstance(action.default, _basestring): + if isinstance(action.default, basestring): default = self._get_value(action, default) setattr(namespace, action.dest, default) @@ -1765,8 +1744,8 @@ class ArgumentParser(_AttributeHolder, _ActionsContainer): arg_strings_pattern = ''.join(arg_string_pattern_parts) # converts arg strings to the appropriate and then takes the action - seen_actions = _set() - seen_non_default_actions = _set() + seen_actions = set() + seen_non_default_actions = set() def take_action(action, argument_strings, option_string=None): seen_actions.add(action) @@ -2179,7 +2158,7 @@ class ArgumentParser(_AttributeHolder, _ActionsContainer): value = action.const else: value = action.default - if isinstance(value, _basestring): + if isinstance(value, basestring): value = self._get_value(action, value) self._check_value(action, value) diff --git a/Lib/test/test_argparse.py b/Lib/test/test_argparse.py index e1bc83a..7ce2b47 100644 --- a/Lib/test/test_argparse.py +++ b/Lib/test/test_argparse.py @@ -22,29 +22,9 @@ import unittest import warnings import argparse -from test import test_support - -try: - from StringIO import StringIO -except ImportError: - from io import StringIO - -try: - set -except NameError: - from sets import Set as set - -try: - sorted -except NameError: - - def sorted(iterable, reverse=False): - result = list(iterable) - result.sort() - if reverse: - result.reverse() - return result +from StringIO import StringIO +from test import test_support class TestCase(unittest.TestCase): |