Message230911
Serhiy, about your 'huge patch' to modernize code:
I am more positive than some because:
1) To me, a one-time gentile change is not 'churning'.
2) As we link to many, most, or even all python-coded stdlib modules (I think there is a proposal for 'all'), there is more benefit to using modern idioms.
On the other hand, 'huge' patches can be too much to discuss, justify, and review all at once.
Using {.. } for sets consistently is a nice-sized chunk to consider. We can identify, discuss, and decide on each sub-case (I have identified 4 so far). It has the additional benefit of being a performance enhancement.
---
'set((...' is used in distutils (which I will not change) and in many tests. So that is not an issue. 'frozenset((' is used 5 times in regular module code. |
|
Date |
User |
Action |
Args |
2014-11-09 20:46:27 | terry.reedy | set | recipients:
+ terry.reedy, rhettinger, ezio.melotti, serhiy.storchaka |
2014-11-09 20:46:27 | terry.reedy | set | messageid: <1415565987.48.0.141666617348.issue22823@psf.upfronthosting.co.za> |
2014-11-09 20:46:27 | terry.reedy | link | issue22823 messages |
2014-11-09 20:46:27 | terry.reedy | create | |
|