Message144522
On 09/25/2011 01:50 AM, Graylin Kim wrote:
>
> Graylin Kim<graylin.kim@gmail.com> added the comment:
>
> 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.
In general brevity is good, but I agree that this is just a style
question, and not very important here.
This wasn't my intention, I was only saying that due to this bug the
wrapping uses fixed width, but I'm hoping that #13041 will be
successfully resolved. |
|
Date |
User |
Action |
Args |
2011-09-25 10:07:24 | zbysz | set | recipients:
+ zbysz, bethard, eric.araujo, denilsonsa, GraylinKim |
2011-09-25 10:07:23 | zbysz | link | issue12806 messages |
2011-09-25 10:07:22 | zbysz | create | |
|