Author Topic: Slow scrolling? [Ubuntu]  (Read 89911 times)

Offline Jenna

  • Administrator
  • Lives here!
  • *****
  • Posts: 7255
Re: Slow scrolling? [Ubuntu]
« Reply #60 on: July 24, 2012, 01:54:04 pm »
As far as I see, it works fine on windows 7 with wx2.8 and actual trunk.

By the way:
the patch I attached above works flawlessly with my tortoise 1.7.7, so it might be (another) issue of SmartSVN with patches.

Offline dmoore

  • Developer
  • Lives here!
  • *****
  • Posts: 1576
Re: Slow scrolling? [Ubuntu]
« Reply #61 on: July 24, 2012, 03:17:36 pm »
As far as I see, it works fine on windows 7 with wx2.8 and actual trunk.

Any thoughts on removing the ticket #9057 junk?

Also, the advantage of patching wx instead of scintilla is at least the wx guys are likely to accept the patch in the next 5 years. :)

Offline Jenna

  • Administrator
  • Lives here!
  • *****
  • Posts: 7255
Re: Slow scrolling? [Ubuntu]
« Reply #62 on: July 24, 2012, 03:28:27 pm »
As far as I see, it works fine on windows 7 with wx2.8 and actual trunk.

Any thoughts on removing the ticket #9057 junk?

Also, the advantage of patching wx instead of scintilla is at least the wx guys are likely to accept the patch in the next 5 years. :)
I will open a ticket there or probably reopen #9057 this evening.

And as far as I can see the "workaround" is no longer needed and can therefore be removed.

Other devs:
any objections against doing it ?
Removing the #9057 "workaround" and the problematic call to update ?

If not I will commit it this evening.

If the whole C::B breaks down  :P , we can still revert it.

Offline Jenna

  • Administrator
  • Lives here!
  • *****
  • Posts: 7255
Re: Slow scrolling? [Ubuntu]
« Reply #63 on: July 30, 2012, 12:19:00 am »

Offline Kalith

  • Multiple posting newcomer
  • *
  • Posts: 67
Re: Slow scrolling? [Ubuntu]
« Reply #64 on: July 30, 2012, 02:44:13 am »
That is good news! Does that mean we'll see these patches applied in the next nightly?

Offline Jenna

  • Administrator
  • Lives here!
  • *****
  • Posts: 7255
Re: Slow scrolling? [Ubuntu]
« Reply #65 on: July 30, 2012, 08:04:08 am »
That is good news! Does that mean we'll see these patches applied in the next nightly?
We do not use wxSTC, but a more or less patrched version based on it.

The changes are in our sources since a week and will therefore be in the next nightly, or if you use my debian or RedHat/CentOS repo you have it there already.

Offline Kalith

  • Multiple posting newcomer
  • *
  • Posts: 67
Re: Slow scrolling? [Ubuntu]
« Reply #66 on: July 30, 2012, 02:29:57 pm »
I can confirm the two patches work very well. Thank you :)

Offline secks

  • Multiple posting newcomer
  • *
  • Posts: 20
Re: Slow scrolling? [Ubuntu]
« Reply #67 on: July 30, 2012, 08:00:36 pm »
the patches did speed it up a bit .. instead of getting 20 some lines i now get around 100 per full-wheel-scroll :) a coworker here was only scrolling 3 or so lines lol .. his bumped up to around 30 now .. weeeeeeeeeeee!  thanks for taking time on this one everyone

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: Slow scrolling? [Ubuntu]
« Reply #68 on: September 02, 2012, 04:18:50 am »
It's basically the same, it avoids the call of wxWindows::Update() on every scrol, which immediately updates the whole invalidated area.
But the original scintilla Redraw() invalidates the whole client area and the other one (most likely) only the scrolled part.
It seems that matching brace highlights, which should be invalidated (and redrawn) no longer are consistently.
Code
int main()
{
    return 0;
}|
Place the cursor where the pipe symbol is, then press the up arrow key once.  The first curly brace remains highlighted.

Could this be related to the changes discussed in this thread?

(Ubuntu LTS 32bit; svn 8328)

Offline Jenna

  • Administrator
  • Lives here!
  • *****
  • Posts: 7255
Re: Slow scrolling? [Ubuntu]
« Reply #69 on: September 02, 2012, 08:54:00 am »
It's basically the same, it avoids the call of wxWindows::Update() on every scrol, which immediately updates the whole invalidated area.
But the original scintilla Redraw() invalidates the whole client area and the other one (most likely) only the scrolled part.
It seems that matching brace highlights, which should be invalidated (and redrawn) no longer are consistently.
Code
int main()
{
    return 0;
}|
Place the cursor where the pipe symbol is, then press the up arrow key once.  The first curly brace remains highlighted.

Could this be related to the changes discussed in this thread?

(Ubuntu LTS 32bit; svn 8328)
I can not reproduce it here.

Offline Alpha

  • Developer
  • Lives here!
  • *****
  • Posts: 1513
Re: Slow scrolling? [Ubuntu]
« Reply #70 on: September 03, 2012, 01:55:28 am »
I can not reproduce it here.
Code
word word word word word word word word

word word word word word


word word word word word word word word


word word


word word word
Select the last "word" - all of the words should be highlighted (this is normal).  Press the up arrow key once.  On Windows Vista; svn 8330, all the words except the last three remain highlighted.

If no one can duplicate it, maybe something is wrong with my computer.  (However, I do not believe this is the case.  These problems are not present in my tests of svn 8133.)

Offline dmoore

  • Developer
  • Lives here!
  • *****
  • Posts: 1576
Re: Slow scrolling? [Ubuntu]
« Reply #71 on: September 03, 2012, 03:43:41 am »
I can not reproduce it here.
Code
... word ...
Select the last "word" - all of the words should be highlighted (this is normal).  Press the up arrow key once.  On Windows Vista; svn 8330, all the words except the last three remain highlighted.

If no one can duplicate it, maybe something is wrong with my computer.  (However, I do not believe this is the case.  These problems are not present in my tests of svn 8133.)
I confirm (ubuntu 12.04, latest trunk). Why do these features rely on the update call and other feature not?
« Last Edit: September 03, 2012, 03:50:58 am by dmoore »

Offline Jenna

  • Administrator
  • Lives here!
  • *****
  • Posts: 7255
Re: Slow scrolling? [Ubuntu]
« Reply #72 on: September 04, 2012, 09:17:02 pm »
If I move the caret away from the selected word, the word gets unselected and all the other highlighted words are shown normal.

Offline dmoore

  • Developer
  • Lives here!
  • *****
  • Posts: 1576
Re: Slow scrolling? [Ubuntu]
« Reply #73 on: September 04, 2012, 09:57:40 pm »
If I move the caret away from the selected word, the word gets unselected and all the other highlighted words are shown normal.

Well, I still have the problem, but it has nothing to do with the small patch you applied here. I tried reverting the patch and it made no difference. Also tried from a fresh config and still have the problem. Note that the problem only occurs when you move the cursor with either keyboard or mouse, but don't scroll. As soon as you scroll the full screen refreshes.

Offline dmoore

  • Developer
  • Lives here!
  • *****
  • Posts: 1576
Re: Slow scrolling? [Ubuntu]
« Reply #74 on: September 04, 2012, 10:52:50 pm »
This patch might do the trick, but not sure if that kills performance on Linux again

Code
Index: src/sdk/cbeditor.cpp
===================================================================
--- src/sdk/cbeditor.cpp (revision 8351)
+++ src/sdk/cbeditor.cpp (working copy)
@@ -502,6 +504,8 @@
         // Set Styling:
         // clear all style indications set in a previous run (is also done once after text gets unselected)
         m_pOwner->GetControl()->IndicatorClearRange(0, eof);
+        m_pOwner->GetControl()->Refresh();
 
         // check that feature is enabled,
         // selected text has a minimal length of 3 and contains no spaces