From 10cf21802d2417c794630f7642a128b757b0a5cb Mon Sep 17 00:00:00 2001 From: Neal Norwitz Date: Thu, 17 Apr 2003 23:09:08 +0000 Subject: [PATCH] Use True in a few more places. Use isinstance(somestring, basestring) instead of type() as per PEP 8 --- Doc/lib/libshlex.tex | 2 +- Lib/shlex.py | 12 +++++------- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/Doc/lib/libshlex.tex b/Doc/lib/libshlex.tex index a79e954008d..9ebe16b36bf 100644 --- a/Doc/lib/libshlex.tex +++ b/Doc/lib/libshlex.tex @@ -177,7 +177,7 @@ includes just \character{"} by default. \end{memberdesc} \begin{memberdesc}{whitespace_split} -If true, tokens will only be split in whitespaces. This is useful, for +If \code{True}, tokens will only be split in whitespaces. This is useful, for example, for parsing command lines with \class{shlex}, getting tokens in a similar way to shell arguments. \versionadded{2.3} diff --git a/Lib/shlex.py b/Lib/shlex.py index ca9c557fead..dd104472f69 100644 --- a/Lib/shlex.py +++ b/Lib/shlex.py @@ -10,8 +10,6 @@ import os.path import sys -from types import StringTypes - try: from cStringIO import StringIO except ImportError: @@ -22,7 +20,7 @@ class shlex: "A lexical analyzer class for simple shell-like syntaxes." def __init__(self, instream=None, infile=None, posix=False): - if type(instream) in StringTypes: + if isinstance(instream, basestring): instream = StringIO(instream) if instream is not None: self.instream = instream @@ -65,7 +63,7 @@ def push_token(self, tok): def push_source(self, newstream, newfile=None): "Push an input source onto the lexer's input source stack." - if type(newstream) in StringTypes: + if isinstance(newstream, basestring): newstream = StringIO(newstream) self.filestack.insert(0, (self.infile, self.instream, self.lineno)) self.infile = newfile @@ -122,7 +120,7 @@ def get_token(self): def read_token(self): quoted = False escapedstate = ' ' - while 1: + while True: nextchar = self.instream.read(1) if nextchar == '\n': self.lineno = self.lineno + 1 @@ -252,7 +250,7 @@ def sourcehook(self, newfile): if newfile[0] == '"': newfile = newfile[1:-1] # This implements cpp-like semantics for relative-path inclusion. - if type(self.infile) in StringTypes and not os.path.isabs(newfile): + if isinstance(self.infile, basestring) and not os.path.isabs(newfile): newfile = os.path.join(os.path.dirname(self.infile), newfile) return (newfile, open(newfile, "r")) @@ -273,7 +271,7 @@ def next(self): raise StopIteration return token -def split(s, posix=1, spaces=1): +def split(s, posix=True, spaces=True): lex = shlex(s, posix=posix) lex.whitespace_split = spaces return list(lex)