diff options
| -rw-r--r-- | Lib/imghdr.py | 27 | ||||
| -rw-r--r-- | Lib/sndhdr.py | 92 | 
2 files changed, 61 insertions, 58 deletions
diff --git a/Lib/imghdr.py b/Lib/imghdr.py index f34f234..d0f66e5 100644 --- a/Lib/imghdr.py +++ b/Lib/imghdr.py @@ -36,14 +36,14 @@ tests = []  def test_rgb(h, f):      """SGI image library""" -    if h[:2] == '\001\332': +    if h[:2] == b'\001\332':          return 'rgb'  tests.append(test_rgb)  def test_gif(h, f):      """GIF ('87 and '89 variants)""" -    if h[:6] in ('GIF87a', 'GIF89a'): +    if h[:6] in (b'GIF87a', b'GIF89a'):          return 'gif'  tests.append(test_gif) @@ -51,7 +51,7 @@ tests.append(test_gif)  def test_pbm(h, f):      """PBM (portable bitmap)"""      if len(h) >= 3 and \ -        h[0] == 'P' and h[1] in '14' and h[2] in ' \t\n\r': +        h[0] == ord('P') and h[1] in b'14' and h[2] in b' \t\n\r':          return 'pbm'  tests.append(test_pbm) @@ -59,7 +59,7 @@ tests.append(test_pbm)  def test_pgm(h, f):      """PGM (portable graymap)"""      if len(h) >= 3 and \ -        h[0] == 'P' and h[1] in '25' and h[2] in ' \t\n\r': +        h[0] == ord('P') and h[1] in b'25' and h[2] in b' \t\n\r':          return 'pgm'  tests.append(test_pgm) @@ -67,28 +67,28 @@ tests.append(test_pgm)  def test_ppm(h, f):      """PPM (portable pixmap)"""      if len(h) >= 3 and \ -        h[0] == 'P' and h[1] in '36' and h[2] in ' \t\n\r': +        h[0] == ord('P') and h[1] in b'36' and h[2] in b' \t\n\r':          return 'ppm'  tests.append(test_ppm)  def test_tiff(h, f):      """TIFF (can be in Motorola or Intel byte order)""" -    if h[:2] in ('MM', 'II'): +    if h[:2] in (b'MM', b'II'):          return 'tiff'  tests.append(test_tiff)  def test_rast(h, f):      """Sun raster file""" -    if h[:4] == '\x59\xA6\x6A\x95': +    if h[:4] == b'\x59\xA6\x6A\x95':          return 'rast'  tests.append(test_rast)  def test_xbm(h, f):      """X bitmap (X10 or X11)""" -    s = '#define ' +    s = b'#define '      if h[:len(s)] == s:          return 'xbm' @@ -96,26 +96,26 @@ tests.append(test_xbm)  def test_jpeg(h, f):      """JPEG data in JFIF format""" -    if h[6:10] == 'JFIF': +    if h[6:10] == b'JFIF':          return 'jpeg'  tests.append(test_jpeg)  def test_exif(h, f):      """JPEG data in Exif format""" -    if h[6:10] == 'Exif': +    if h[6:10] == b'Exif':          return 'jpeg'  tests.append(test_exif)  def test_bmp(h, f): -    if h[:2] == 'BM': +    if h[:2] == b'BM':          return 'bmp'  tests.append(test_bmp)  def test_png(h, f): -    if h[:8] == "\211PNG\r\n\032\n": +    if h[:8] == b'\211PNG\r\n\032\n':          return 'png'  tests.append(test_png) @@ -159,3 +159,6 @@ def testall(list, recursive, toplevel):                  print(what(filename))              except IOError:                  print('*** not found ***') + +if __name__ == '__main__': +    test() diff --git a/Lib/sndhdr.py b/Lib/sndhdr.py index f505804..aa9b3b9 100644 --- a/Lib/sndhdr.py +++ b/Lib/sndhdr.py @@ -30,23 +30,23 @@ explicitly given directories.  # The file structure is top-down except that the test program and its  # subroutine come last. -__all__ = ["what","whathdr"] +__all__ = ['what', 'whathdr']  def what(filename): -    """Guess the type of a sound file""" +    """Guess the type of a sound file."""      res = whathdr(filename)      return res  def whathdr(filename): -    """Recognize sound headers""" -    f = open(filename, 'rb') -    h = f.read(512) -    for tf in tests: -        res = tf(h, f) -        if res: -            return res -    return None +    """Recognize sound headers.""" +    with open(filename, 'rb') as f: +        h = f.read(512) +        for tf in tests: +            res = tf(h, f) +            if res: +                return res +        return None  #-----------------------------------# @@ -57,12 +57,12 @@ tests = []  def test_aifc(h, f):      import aifc -    if h[:4] != 'FORM': +    if h[:4] != b'FORM':          return None -    if h[8:12] == 'AIFC': +    if h[8:12] == b'AIFC':          fmt = 'aifc' -    elif h[8:12] == 'AIFF': -        fmt = 'aiff' +    elif h[8:12] == b'AIFF': +        fmt = b'aiff'      else:          return None      f.seek(0) @@ -70,25 +70,25 @@ def test_aifc(h, f):          a = aifc.openfp(f, 'r')      except (EOFError, aifc.Error):          return None -    return (fmt, a.getframerate(), a.getnchannels(), \ -            a.getnframes(), 8*a.getsampwidth()) +    return (fmt, a.getframerate(), a.getnchannels(), +            a.getnframes(), 8 * a.getsampwidth())  tests.append(test_aifc)  def test_au(h, f): -    if h[:4] == '.snd': -        f = get_long_be -    elif h[:4] in ('\0ds.', 'dns.'): -        f = get_long_le +    if h[:4] == b'.snd': +        func = get_long_be +    elif h[:4] in (b'\0ds.', b'dns.'): +        func = get_long_le      else:          return None -    type = 'au' -    hdr_size = f(h[4:8]) -    data_size = f(h[8:12]) -    encoding = f(h[12:16]) -    rate = f(h[16:20]) -    nchannels = f(h[20:24]) +    filetype = 'au' +    hdr_size = func(h[4:8]) +    data_size = func(h[8:12]) +    encoding = func(h[12:16]) +    rate = func(h[16:20]) +    nchannels = func(h[20:24])      sample_size = 1 # default      if encoding == 1:          sample_bits = 'U' @@ -100,26 +100,26 @@ def test_au(h, f):      else:          sample_bits = '?'      frame_size = sample_size * nchannels -    return type, rate, nchannels, data_size/frame_size, sample_bits +    return filetype, rate, nchannels, data_size / frame_size, sample_bits  tests.append(test_au)  def test_hcom(h, f): -    if h[65:69] != 'FSSD' or h[128:132] != 'HCOM': +    if h[65:69] != b'FSSD' or h[128:132] != b'HCOM':          return None      divisor = get_long_be(h[128+16:128+20]) -    return 'hcom', 22050/divisor, 1, -1, 8 +    return 'hcom', 22050 / divisor, 1, -1, 8  tests.append(test_hcom)  def test_voc(h, f): -    if h[:20] != 'Creative Voice File\032': +    if h[:20] != b'Creative Voice File\032':          return None      sbseek = get_short_le(h[20:22])      rate = 0 -    if 0 <= sbseek < 500 and h[sbseek] == '\1': +    if 0 <= sbseek < 500 and h[sbseek] == b'\1':          ratecode = ord(h[sbseek+4])          rate = int(1000000.0 / (256 - ratecode))      return 'voc', rate, 1, -1, 8 @@ -129,7 +129,7 @@ tests.append(test_voc)  def test_wav(h, f):      # 'RIFF' <len> 'WAVE' 'fmt ' <len> -    if h[:4] != 'RIFF' or h[8:12] != 'WAVE' or h[12:16] != 'fmt ': +    if h[:4] != b'RIFF' or h[8:12] != b'WAVE' or h[12:16] != b'fmt ':          return None      style = get_short_le(h[20:22])      nchannels = get_short_le(h[22:24]) @@ -141,7 +141,7 @@ tests.append(test_wav)  def test_8svx(h, f): -    if h[:4] != 'FORM' or h[8:12] != '8SVX': +    if h[:4] != b'FORM' or h[8:12] != b'8SVX':          return None      # Should decode it to get #channels -- assume always 1      return '8svx', 0, 1, 0, 8 @@ -150,7 +150,7 @@ tests.append(test_8svx)  def test_sndt(h, f): -    if h[:5] == 'SOUND': +    if h[:5] == b'SOUND':          nsamples = get_long_le(h[8:12])          rate = get_short_le(h[20:22])          return 'sndt', rate, 1, nsamples, 8 @@ -159,7 +159,7 @@ tests.append(test_sndt)  def test_sndr(h, f): -    if h[:2] == '\0\0': +    if h[:2] == b'\0\0':          rate = get_short_le(h[2:4])          if 4000 <= rate <= 25000:              return 'sndr', rate, 1, -1, 8 @@ -167,21 +167,21 @@ def test_sndr(h, f):  tests.append(test_sndr) -#---------------------------------------------# -# Subroutines to extract numbers from strings # -#---------------------------------------------# +#-------------------------------------------# +# Subroutines to extract numbers from bytes # +#-------------------------------------------# -def get_long_be(s): -    return (ord(s[0])<<24) | (ord(s[1])<<16) | (ord(s[2])<<8) | ord(s[3]) +def get_long_be(b): +    return (b[0] << 24) | (b[1] << 16) | (b[2] << 8) | b[3] -def get_long_le(s): -    return (ord(s[3])<<24) | (ord(s[2])<<16) | (ord(s[1])<<8) | ord(s[0]) +def get_long_le(b): +    return (b[3] << 24) | (b[2] << 16) | (b[1] << 8) | b[0] -def get_short_be(s): -    return (ord(s[0])<<8) | ord(s[1]) +def get_short_be(b): +    return (b[0] << 8) | b[1] -def get_short_le(s): -    return (ord(s[1])<<8) | ord(s[0]) +def get_short_le(b): +    return (b[1] << 8) | b[0]  #--------------------#  | 
