User forums > Using Code::Blocks
Slow scrolling? [Ubuntu]
Jenna:
What makes me wonder, is that IncrementalSearch still works, but HighlightOccurrences does not.
And the ClearRange code is the same in both cases.
We can do the Refresh as workaround for the moment, but it looks like there must be some deeper issue that should be fixed, before it bites us at another place.
At the moment I would prefer not to change the (wx)scintilla code, if it is not definitely caused by it.
But (as I posted) before, HighlightOccurrences work with wxSTC and IncrementaSearch works with "our" wxScintilla.
So it might still be something in ouzr code, probably in our (wx)scintilla-changes.
Jenna:
--- Quote from: dmoore on September 05, 2012, 04:55:44 pm ---btw, either approach seems to work fine on my netbook (doesn't slow things down too much). I would still advocate reducing the number of ClearRange calls, because that could be expensive on a really big file.
I don't understand why this wouldn't also affect the wx demo.
--- End quote ---
ClearRange is only called once for every selection change, so it does not happen too often.
dmoore:
--- Quote from: jens on September 05, 2012, 04:58:59 pm ---ClearRange is only called once for every selection change, so it does not happen too often.
--- End quote ---
probably right, but scrolling by holding the down arrow on a really big file could have a decent sized performance hit. (think of the children... err... netbooks)
dmoore:
--- Quote from: jens on September 05, 2012, 04:57:47 pm ---What makes me wonder, is that IncrementalSearch still works, but HighlightOccurrences does not.
And the ClearRange code is the same in both cases.
--- End quote ---
because (painstate==notpainting) == true for incremental search. IMHO it's a scintilla bug.
dmoore:
So how about this:
1. Use the refresh workaround wherever it is needed
2. File a bug with scintilla (checking their bug tracker and sources first)
I will do some testing of the performance of the clear indicator calls on a large file on my netbook and propose a patch if necessary. (I used to have highlighting switched off because of the scrolling performance but it may not be so bad after the scrolling patch.)
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version