Bug 505777 - Introduce notification manager API
Summary: Introduce notification manager API
Status: RESOLVED FIXED
Alias: None
Product: Handly
Classification: Technology
Component: Core (show other bugs)
Version: 0.5   Edit
Hardware: All All
: P3 enhancement
Target Milestone: 0.6   Edit
Assignee: Vladimir Piskarev CLA
QA Contact:
URL:
Whiteboard:
Keywords: api
Depends on:
Blocks:
 
Reported: 2016-10-12 06:42 EDT by Vladimir Piskarev CLA
Modified: 2016-10-12 08:53 EDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Vladimir Piskarev CLA 2016-10-12 06:42:41 EDT 
Current model implementations contain very similar code in their source file's hWorkingCopyModeChanged() and NotifyingReconcileOperation. Seemingly, this code could be generalized at the SourceFile level by introducing INotificationManager API. The only thing that model implementations will have to do to use this facility is to register an implementation of the notification manager in the model context.
Comment 1 Vladimir Piskarev CLA 2016-10-12 08:53:09 EDT 
Pushed to master:
http://git.eclipse.org/c/handly/org.eclipse.handly.git/commit/?id=bb22a389ecf08fba622d8ac70a5d30e1f637dded


New API:

* INotificationManager
* NotificationManager
* ElementChangeListenerList
* ElementDelta$Factory
* SourceFile$NotifyingReconcileOperation