python
diff --git a/‎Lib/argparse.py
Lines changed: 4 additions & 6 deletions b/‎Lib/argparse.py
Lines changed: 4 additions & 6 deletions
diff --git a/‎Lib/test/test_argparse.py
Lines changed: 1 addition & 1 deletion b/‎Lib/test/test_argparse.py
Lines changed: 1 addition & 1 deletion
diff --git a/‎Misc/NEWS.d/next/Library/2024-04-19-05-58-50.gh-issue-117766.J3xepp.rst
Lines changed: 1 addition & 0 deletions b/‎Misc/NEWS.d/next/Library/2024-04-19-05-58-50.gh-issue-117766.J3xepp.rst
Lines changed: 1 addition & 0 deletions
@@ -584,8 +584,7 @@ def _metavar_formatter(self, action, default_metavar):
         if action.metavar is not None:
             result = action.metavar
         elif action.choices is not None:
-            choice_strs = [str(choice) for choice in action.choices]
-            result = '{%s}' % ','.join(choice_strs)
+            result = '{%s}' % ','.join(map(str, action.choices))
         else:
             result = default_metavar
 
@@ -633,8 +632,7 @@ def _expand_help(self, action):
             if hasattr(params[name], '__name__'):
                 params[name] = params[name].__name__
         if params.get('choices') is not None:
-            choices_str = ', '.join([str(c) for c in params['choices']])
-            params['choices'] = choices_str
+            params['choices'] = ', '.join(map(str, params['choices']))
         return self._get_help_string(action) % params
 
     def _iter_indented_subactions(self, action):
@@ -743,7 +741,7 @@ def _get_action_name(argument):
     elif argument.dest not in (None, SUPPRESS):
         return argument.dest
     elif argument.choices:
-        return '{' + ','.join(argument.choices) + '}'
+        return '{%s}' % ','.join(map(str, argument.choices))
     else:
         return None
 
@@ -2613,7 +2611,7 @@ def _check_value(self, action, value):
         # converted value must be one of the choices (if specified)
         if action.choices is not None and value not in action.choices:
             args = {'value': value,
-                    'choices': ', '.join(map(repr, action.choices))}
+                    'choices': ', '.join(map(str, action.choices))}
             msg = _('invalid choice: %(value)r (choose from %(choices)s)')
             raise ArgumentError(action, msg % args)
 
 
@@ -2361,7 +2361,7 @@ def test_wrong_argument_subparsers_no_destination_error(self):
             parser.parse_args(('baz',))
         self.assertRegex(
             excinfo.exception.stderr,
-            r"error: argument {foo,bar}: invalid choice: 'baz' \(choose from 'foo', 'bar'\)\n$"
+            r"error: argument {foo,bar}: invalid choice: 'baz' \(choose from foo, bar\)\n$"
         )
 
     def test_optional_subparsers(self):
 
@@ -0,0 +1 @@
+Always use :func:`str()` to print ``choices`` in :mod:`argparse`.
Original file line number	Diff line number	Diff line change
`@@ -2361,7 +2361,7 @@ def test_wrong_argument_subparsers_no_destination_error(self):`
`2361`	`2361`	`parser.parse_args(('baz',))`
`2362`	`2362`	`self.assertRegex(`
`2363`	`2363`	`excinfo.exception.stderr,`
`2364`		`- r"error: argument {foo,bar}: invalid choice: 'baz' \(choose from 'foo', 'bar'\)\n$"`
	`2364`	`+ r"error: argument {foo,bar}: invalid choice: 'baz' \(choose from foo, bar\)\n$"`
`2365`	`2365`	`)`
`2366`	`2366`
`2367`	`2367`	`def test_optional_subparsers(self):`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	+Always use :func:`str()` to print ``choices`` in :mod:`argparse`.