On Tue, Feb 19, 2013 at 3:08 PM, Michael Hanselmann <[email protected]> wrote: > 2013/2/15 Guido Trotter <[email protected]>: >>> + if translate is None: >>> + fn = _CompileHandlerPath >>> + else: >>> + fn = translate >> >> How about calling it translate_fn >> (or just translate, if you prefer). >> fn is very generic as a name. > > Ack, but it's used 43 times and makes lines much longer (almost an > additional 0.5 kB with a “translate_” prefix! :-)). It's a local > helper variable—can we keep “fn”? >
Since when we code for source code space optimization? :) Thanks, Guido
