[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[T-Merge] wrap long lines issues a comments

From: Oto BREZINA <otik_at_printflow.eu>
Date: Tue, 19 Apr 2011 21:20:34 +0200

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.

-- 
Oto BREZINA, Printflow s.r.o., EU
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2721520
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-04-19 21:20:46 CEST

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.