Re: [T-Merge Patch] Lock build to reduce build time

On 28.04.2011 12:28, Oto BREZINA wrote:
> On 2011-04-28 12:23, Stefan Küng wrote:
>> On Thu, Apr 28, 2011 at 09:30, Oto BREZINA<otik_at_printflow.eu> wrote:
>>>
>>> Sounds good to me, I did not want to make that big change just improve
>>> actual code. Of course real solutions are better then hacks.
>>> In fact "all" static attributes may be refactored out.
>>> However this vector is needed for updating scrolls and other stuffs, so
>>> not sure how it can be implemented properly ...
>>
>> class Screen2View
>> {
>> public:
>> int GetViewLine(int screenline);
>> private:
>> void RebuildVector();
>> vector<> m_screen2view;
>> };
>>
>> int GetViewLine(int screenline)
>> {
>> if (needsrebuild)
>> {
>> RebuildVector();
>> needrebuild = false;
>> }
>> return m_screen2View[screenline];
>> }
>>
>>
>> basically, you hide the vector completely from the outside. Not sure
>> why you think that updating scrollbars is somehow related to this?
> Because when I have not scroll bars and locator with delayed build it
> does not work. If you make it working it would be great.

Should work as of r21234.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2725514
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].