gh-96548: argparse: Remove unused name variable when handling ArgumentTypeError

[JonathonReinhart]

This removes the unused name variable in the block where ArgumentTypeError is handled.

ArgumentTypeError errors are handled by showing just the string of the exception; unlike ValueError, the name (__name__) of the function is not included in the error message.

Fixes #96548

• Issue: argparse: 'name' variable unused when handling ArgumentTypeError #96548

[bedevere-bot]

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

[ghost]

All commit authors signed the Contributor License Agreement.

[JonathonReinhart]

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

In my opinion, this fix is too trivial to warrant a NEWS entry. There is no user-visible change in behavior whatsoever.

[JelleZijlstra]

Looks good, thanks. Going to change this in main only since it doesn't seem like it could cause a user-visible bug.

[JonathonReinhart]

Looks good, thanks. Going to change this in main only since it doesn't seem like it could cause a user-visible bug.

Agreed, this is just a code cleanliness thing.

FWIW I discovered this while looking at the code to understand how exceptions from add_argument(type=) callbacks are handled. I noticed that when ValueError is raised, the __name__ of the function is used in the error message. And I was checking to see if the same applied to ArgumentTypeError.