Message109074
Stylistically, it would be nice to eliminate the local variable reassignment entirely:
self.Emin = DefaultContext.Emin if Emin is None else Emin
self.Emax = DefaultContext.Emax if Emax is None else Emax
self._ignored_flags = [] if _ignored_flags is None else _ignored_flags
Also, to keep the code consistent between versions, it may be better to avoid set/dict comprehensions and stick with the old:
dict([(s, int(s in flags)) for s in _signals])
or slightly more modern generator comprehension:
dict((s, int(s in flags)) for s in _signals) |
|
Date |
User |
Action |
Args |
2010-07-01 21:16:48 | rhettinger | set | recipients:
+ rhettinger, mark.dickinson, ezio.melotti, ubershmekel, skrah |
2010-07-01 21:16:48 | rhettinger | set | messageid: <1278019008.16.0.889192745156.issue9136@psf.upfronthosting.co.za> |
2010-07-01 21:16:46 | rhettinger | link | issue9136 messages |
2010-07-01 21:16:45 | rhettinger | create | |
|