Author bethard
Recipients SilentGhost, Tarsis.Azevedo, akira, bethard, doughellmann, eric.araujo
Date 2010-11-27.14:46:40
SpamBayes Score 9.30603e-06
Marked as misclassified No
Message-id <>
Tried to comment in Rietveld but it didn't work for some reason. Anyway, I think the patch isn't good - changing the type error message to "'invalid %s value: %r details: "%s"'" will change the behavior of a bunch of programs, and it's not clearly for the better. Instead, you should raise an ArgumentTypeError instead of a ValueError, and give it whatever message you want there. That is, let's keep this patch local to the FileType, and not touch the rest of argparse.
Date User Action Args
2010-11-27 14:46:44bethardsetrecipients: + bethard, eric.araujo, doughellmann, SilentGhost, akira, Tarsis.Azevedo
2010-11-27 14:46:44bethardsetmessageid: <>
2010-11-27 14:46:40bethardlinkissue9509 messages
2010-11-27 14:46:40bethardcreate