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
> 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.
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-04-19 22:39:59 CEST
This is an archived mail posted to the TortoiseSVN Dev