Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 351754

Summary: Duplicate ProgressMonitorBridge class
Product: z_Archived Reporter: Konstantin Komissarchik <konstantin>
Component: SapphireAssignee: Konstantin Komissarchik <konstantin>
Status: REOPENED --- QA Contact:
Severity: normal    
Priority: P3    
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:

Description Konstantin Komissarchik CLA 2011-07-11 16:44:20 EDT 
A ProgressMonitorBridge class exists in the following two plugins:

org.eclipse.sapphire.platform
org.eclipse.sapphire.ui

The two classes have similar purpose, but not exactly the same implementation. The one in the UI plugin should be eliminated.
Comment 1 Konstantin Komissarchik CLA 2011-07-11 16:48:01 EDT 
Fixed.
Comment 2 Konstantin Komissarchik CLA 2011-07-11 17:27:51 EDT 
It turns out that fixing this is harder than it looks. Adding a dependency from sapphire.ui to sapphire.platform causes a cycle in between features which breaks the commandline build. Reverting the change and deferring to a later release.
Comment 3 Konstantin Komissarchik CLA 2012-04-24 15:17:36 EDT 
To close this issue, I removed the copy of this class in org.eclipse.sapphire.plugin. The placement may need to be revisited when ui core and swt code are fully separated, but it should be good enough for now.
Comment 4 Konstantin Komissarchik CLA 2012-04-26 11:08:19 EDT 
The copy of ProgressMonitoryBridge class in platform plugin is used by adopters. Those usages are non-UI, so they cannot switch to depend on the copy in ui plugin. Restored the class. Deferring indefinitely.