On Jul 7, 4:23 pm, Mike Orr <[email protected]> wrote: > I don't want to get into a bikeshed argument over changing one local > variable from three letters to nine. I just want to point out that > there is opposition to the change, and that the status quo should get > the benefit of the doubt.
I don't want to make a big deal out of it either. My only argument is that the status quo in this case may cause a problem for someone using the template that spits out routing.py in a new project. If they actually do want to use map() later on as they develop their make_map (), they're going to get a potentially confusing exception that routes.base.Mapping doesn't implement __call__(). If we want to improve Pylons and help people have success with it, side stepping such confusion is arguably a Good Thing (TM), especially since it is just template code and doesn't cause any interface/compatibility problems. As for the length, yes, I actually do use variables like that all the time. My code reads like English, and the people who maintain it love that. :-) --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "pylons-discuss" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/pylons-discuss?hl=en -~----------~----~----~----~------~----~------~--~---
