Erwin Lotter

I ran a few quick tests on some functions that I worked on at a late stage. As expected, they still contain bugs.: Copying in line selection mode copies one extra line. Column selection mode behaves very strange. Moving the cursor around seems to insert characters. And other things. Find will not proceed to the second finding in a line. Regular expression search for "cool" crashes. CoolDemo: Menu entry 'Stream Selection' is disabled. Dropping text to the right of the source point inserts at destination...

branches/elotter-cooledit: Resolved some problems spotted by Claude in [feature-requests:#286]

... or bring your functionality to the trunk, by reimplementing it within the new design and cleaned up implementation. AI can probably do that in a jiffy, though! If AI can manage that, then in my view, it would be the ideal solution. And I have a few more ideas for minor improvements regarding what AI could do next :-).

Wow, that’s a huge task. I’m surprised it’s even possible to extract relevant fixes from the many changes I’ve made. I also recall that some fixes turned out to be insufficient, and I later improved them as part of more extensive overhauls—without the connection to branch 7 being made clear. I wonder if it might be more effective to try to make elotter-cooledit API/ABI compatible with cooledit-7, and thereby replace cooledit-7. However, I have no experience with this, so it’s hard for me to say whether...

branches/elotter-cooledit:

branches/elotter-cooledit:

Since r9105, your horizontal range is CalculateXRange() + GetTotalMarginWidth() / GetCharWidth(). That has no column for the caret, and it rounds the margin down. That is the point I overlooked. Since I corrected this I can see my cursor again :-). Thanks and sorry for the hassle.

Your committed branch doesn't have the [r9103] change yet. True, but as I said: Merging [r9103] into my version of TCoolTextWnd didn't solve the problem ... Although I agree to the arguments in [r9103], it doesn't change anything. I also tried with the modified [r9103] code above, but the problem still exists. @vattila: Can you confirm that with the trunk version the cursor gets not hidden by the vertical bar?