[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: PATCH: Refactor script-column.cc for improved reading and fewer line
From: |
Carl Sorensen |
Subject: |
Re: PATCH: Refactor script-column.cc for improved reading and fewer lines |
Date: |
Fri, 4 Dec 2009 17:35:16 -0700 |
On 12/4/09 5:27 PM, "Carl Sorensen" <address@hidden> wrote:
>
>
>
>
> On 12/4/09 1:31 PM, "David Kastrup" <address@hidden> wrote:
>
>> Carl Sorensen <address@hidden> writes:
>>
>>> On 12/4/09 12:02 PM, "Han-Wen Nienhuys" <address@hidden> wrote:
>>>
>>>> While having these explanations in mail is nice, it would be best to
>>>> put as much of possible of these comments into the code.
>>>
>>> Understood.
>>>
>>> When I see David's suggestions, I'll make sure it gets commented.
>>
>> I just came home and have to go to bed now. I'll write code tomorrow
>> that reflects the logic of your description and embeds it as well. And
>> some part of it probably will have to go into CG or similar, too.
>>
>
>
> I've put a new patch set (Set 3) that makes use of the updating in the for
> loop, as David suggested.
>
> I've eliminated the code that added the difference in script_priority values
> and just use a constant bump in outside_script_priority, which makes the
> code cleaner.
>
> I've also added comments to try to capture the intent.
>
> Reviews are welcome.
Sorry, I forgot the Rietveld link:
http://codereview.appspot.com/166057
Thanks,
Carl
- PATCH: Refactor script-column.cc for improved reading and fewer lines, Carl Sorensen, 2009/12/04
- Re: PATCH: Refactor script-column.cc for improved reading and fewer lines, Trevor Daniels, 2009/12/05
- Re: PATCH: Refactor script-column.cc for improved reading and fewer lines, David Kastrup, 2009/12/05
- Re: PATCH: Refactor script-column.cc for improved reading and fewer lines, Carl Sorensen, 2009/12/05
- Re: PATCH: Refactor script-column.cc for improved reading and fewer lines, Carl Sorensen, 2009/12/05
- Re: PATCH: Refactor script-column.cc for improved reading and fewer lines, Trevor Daniels, 2009/12/05
- Re: PATCH: Refactor script-column.cc for improved reading and fewer lines, Carl Sorensen, 2009/12/05