Re: [T-Merge] wrap long lines issues a comments

On 19.04.2011 21:20, Oto BREZINA wrote:
> After some code reading I start to understand how wrap long lines is
> handled. It seems that it added lots of problems (most are probably
> fixed('workarounded')).
> Still Screen2View and caret, can be improved.
>
> 1. caret should point to view instead of screen coordinates
> + will be preserved over wrapped lines
> + some other operations will be simplified
> - lot of code to rewrite
>
> 2. Screen2view may be struct/class to store more info (subline index,
> first character of subline )
> + would simplify lot of code, thus increase speed a bit
> ! may need good design to do not affect too much code
> - more memory consumption (bit)
>
> I would like to add that current implementaion also make functions like
> CBottomView::UseBothBlocks do more work that they should if wraping get
> effect. There is multi time view processing. I'll fix this.
>
> I would like to hear your opinion about this. I don't know how much time
> is till 1.7.

If you have an idea on how to improve it, then just go for it.
1.7 is still a long way ahead, I doubt it will happen before summer.

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=2721546
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].