Message144513
I fully support taking blank line based line-wrapping approach and agree with Zbyszek's suggested indentation approach as well. I am not sure why they didn't occur to me at the time but they are certainly a more effective and widely adopted approaches to the structured text problem.
I suppose here is where I should volunteer to update the patch file...
Re: Bike-shedding
>dash '-' has special meaning in brackets:
Good catch, I had intended on '-' being a valid list item character. It clearly needs to be escaped. Not that it would matter given your proposed alternative.
>> if(list_match):
>Parenthesis unnecessary.
In my defense I have the sadistic pleasure of coding in PHP where they are necessary for 8 hours a day for my day job. I can only apologize profusely for my offense and beg for forgiveness :)
>> lines = list()
>Why not just 'lines = []'?
Not to get off topic, but I happen to like list() and dict() instead of [] and {} for empty collections. If there are non-religious reasons for avoiding this practice I'll consider it. I don't want to invoke a holy war here, just wondering if there are practical reasons.
>One a side note: due to #13041 the terminal width is normally stuck
at 80 chars.
Not a good reason to remove the flexibility from the implementation I don't think. |
|
Date |
User |
Action |
Args |
2011-09-24 23:50:07 | GraylinKim | set | recipients:
+ GraylinKim, bethard, eric.araujo, zbysz, denilsonsa |
2011-09-24 23:50:07 | GraylinKim | set | messageid: <1316908207.48.0.255883874112.issue12806@psf.upfronthosting.co.za> |
2011-09-24 23:50:06 | GraylinKim | link | issue12806 messages |
2011-09-24 23:50:06 | GraylinKim | create | |
|