Bug 129967 - System Settings being fired too often
Summary: System Settings being fired too often
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Steve Northover CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 128472 129887
  Show dependency tree
 
Reported: 2006-03-01 11:00 EST by Tod Creasey CLA
Modified: 2019-09-06 16:18 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Tod Creasey CLA 2006-03-01 11:00:51 EST
20060228

We are getting the system settings callback too often including when you switch to another application from Eclipse. As we have no context from the Settings (I only care about font changes for instance) we end up offering to restart too often.
Comment 1 Steve Northover CLA 2006-03-01 13:24:39 EST
1) Can you show me?
2) You offer to restart when any setting has changed?  Sounds wrong.  You need to save away the value you care about and test to see whether it has changed.
Comment 2 Steve Northover CLA 2006-03-01 16:34:05 EST
I have released a fix.  Can you test it?

Note that system changes can happen at any time.  For example, sometimes in order to work around a bug, a programe will set a global system value (say a time out), make a call, and then restore the old value.  This will trigger a settings change.

I thought we had a snippet ...
Comment 3 Dani Megert CLA 2006-03-02 01:54:14 EST
>For example, sometimes in
>order to work around a bug, a programe will set a global system value
That's what I pointed out in the Platform UI bug as well. If that restart dialog "feature" is kept in I want at least be able to disable it or get more fine grained control (which AFAIU isn't possible).
Comment 4 Steve Northover CLA 2006-03-03 10:29:43 EST
Reopen if not FIXEDFORYOU.
Comment 5 Steve Northover CLA 2007-12-14 17:49:20 EST
This has been fixed for a while, however, we are planning to make a code change that might bring the bug back.  I tried to make it happen with Tod but his machine changed and the bug didn't happen.

Specifically, SWT.Settings was getting fired when other applications were activated and this no longer seems to be happening.  Dani, did the problem ever happen to you?
Comment 6 Dani Megert CLA 2007-12-17 05:43:18 EST
> Dani, did the problem ever happen to you?
It did (see bug 129887). But after bug 129887 got fixed I never got it again. Even if more settings get fired again it won't have any effect because Platform UI's settings listener now checks the high contrast flag and only shows the restart dialog when it changed.
Comment 7 Tod Creasey CLA 2007-12-17 08:00:20 EST
Dani is right. The only thing we might want to do to expand this is to also check for a change to the system default font but this change won't affect our ability to do that either.
Comment 8 Kevin Barnes CLA 2008-02-12 17:09:11 EST
Steve, should we revisit this? There was a reason for comment #5, but I don't remember what it was. From comment #6 and comment #7 it looks like we can safely change the code.
Comment 9 Steve Northover CLA 2008-02-12 17:44:09 EST
From memory, we changed and released the code, but only on Vista would there be the possibility of too many SWT.Setings events.  Could be wrong about this ...
Comment 10 Eclipse Webmaster CLA 2019-09-06 16:18:35 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.