Bug 236355 - [api] Add an IRSEInteractionProvider#eventExec() method
Summary: [api] Add an IRSEInteractionProvider#eventExec() method
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: 3.0 RC4   Edit
Assignee: Martin Oberhuber CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords: api
Depends on:
Blocks: 237375 227750
  Show dependency tree
 
Reported: 2008-06-10 00:02 EDT by Martin Oberhuber CLA
Modified: 2008-06-24 11:42 EDT (History)
1 user (show)

See Also:
ddykstal.eclipse: review+


Attachments
Patch providing the method (3.34 KB, patch)
2008-06-10 05:33 EDT, Martin Oberhuber CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Oberhuber CLA 2008-06-10 00:02:58 EDT
The RSE ISystemRegistry#fireEvent() methods are documented to switch to the UI Thread (Display.asyncExec()), but only if not on the UI thread already.

In order to abstract from the event handling through IRSEInteractionProvider, we'll need a method eventExec() which has exactly these semantics:

/**
 * Run a Runnable in the message Thread.
 * Directly runs if already on the right Thread, or does an asyncExec()
 * otherwise.
 */
public void eventExec(Runnable r);
Comment 1 Martin Oberhuber CLA 2008-06-10 05:33:33 EDT
Created attachment 104294 [details]
Patch providing the method
Comment 2 Martin Oberhuber CLA 2008-06-10 05:34:06 EDT
Committed:
[236355] [api] Add an IRSEInteractionProvider#eventExec() method
Comment 3 David Dykstal CLA 2008-06-10 09:26:33 EDT
Should not be a problem adding this since this is a new interface.
Comment 4 Martin Oberhuber CLA 2008-06-10 09:31:26 EDT
We'll need this eventually in order to fix bug 227750.