[Python-Dev] [Python-checkins] cpython: #1097797: Add CP273 codec, and exercise it in the test suite (original) (raw)

A.M. Kuchling amk at amk.ca
Mon Nov 11 03:30:02 CET 2013


On Sun, Nov 10, 2013 at 06:48:30PM -0500, Terry Reedy wrote:

>+ def encode(self,input,errors='strict'): >+ return codecs.charmapencode(input,errors,encodingtable)

You left out the spaces after the call commas (PEP8)

Thanks for noting this. This file is programmatically generated by the scripts in Tools/unicode/ . Many of the files in this directory also have similar PEP8 violations:

grep -r 'def encode' Lib/encodings/ |grep ',inpu' |less ... Lib/encodings//koi8_r.py: def encode(self,input,errors='strict'): Lib/encodings//koi8_u.py: def encode(self,input,errors='strict'): Lib/encodings//mac_arabic.py: def encode(self,input,errors='strict'): Lib/encodings//mac_centeuro.py: def encode(self,input,errors='strict'):

So I'll take a look at fixing the code generator to generate properly formatted code, but it probably isn't worth modifying all of the existing files in Lib/encodings.

--amk



More information about the Python-Dev mailing list