On Sat, Jan 17, 2009 at 4:58 PM, Barry Warsaw <barry at python.org> wrote: > On Jan 16, 2009, at 10:52 PM, Benjamin Peterson wrote: >> On Fri, Jan 16, 2009 at 9:45 PM, Barry Warsaw <barry at python.org> wrote: >>> - - if type(args[0]) is types.StringType: >>> + if type(args[0]) in types.StringTypes: >> >> It'd probably be better to replace that whole line with >> isinstance(args[0], basestring). > > I thought about that, but clearly the style of that module is to use the > 'is' test. I'm assuming that's because of some required backward > compatibility reason, but honestly I didn't check, I just copied the style > of the file. optparse is now no longer externally maintained, so it could probably use a little TLC and modernization. > >>> The fact that 'a' and 'b' are unicodes and not accepted as keyword >>> arguments >>> is probably the tougher problem. I haven't yet looked at what it might >>> take >>> to fix. Is it worth fixing in 2.6 or is this a wait-for-2.7 thing? >> >> Actually, this looks like a one line fix, too: .... > That seems reasonable. I've posted this to the tracker with a test: http://bugs.python.org/issue4978 -- Regards, Benjamin
RetroSearch is an open source project built by @garambo | Open a GitHub Issue
Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo
HTML:
3.2
| Encoding:
UTF-8
| Version:
0.7.4