Attention is currently required from: fixeria, laforge.
1 comment:
Patchset:
CSV matching: I have other use cases than you do. […]
In this new patch set, the name and camel_to_snake() are back (they were lost by accident from branch rebasing).
reasons in my decision against a meta class:
- i don't understand what purpose a meta class serves.
- fixeria's example shows that the shortest way to achieve class attributes is
~~~
class Foo:
a = 1
b = 2
~~~
I like this plain python better than kwargs juggling.
- upcoming patches add a lot of class attributes, and i would like to consistently use one way. I chose the "normal python" way, so as a first step, this patch drops the meta class.
- name: i would like to set explicit names like 'IMSI' instead of 'imsi',
Hence i change to using an explicit 'name' attribute instead of getattr magic.
When no explicit name is set, all still behaves as before this patch.
I hope that I've made my point, so i am now, in a hopeful way, resolving this thread.
Please -1 again if i should change it.
To view, visit change 39741. To unsubscribe, or for help writing mail filters, visit settings.