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

Bug 337830

Summary:	Java Metamodel : org.eclipse.gmt.modisco.java should use MinimalEObjectImpl
Product:	[Modeling] MoDisco	Reporter:	Fabien Giquel <fabien.giquel>
Component:	Technologies	Assignee:	Nicolas Bros <nicolas.bros>
Status:	CLOSED FIXED	QA Contact:
Severity:	normal
Priority:	P2	CC:	hugo.bruneliere, modisco.web-inbox, nicolas.bros
Version:	0.9.0	Flags:	nicolas.bros: indigo+
Target Milestone:	0.9.0 M7
Hardware:	PC
OS:	Windows XP
Whiteboard:

Description Fabien Giquel

2011-02-22 09:08:40 EST

org.eclipse.gmt.modisco.java.classicEObject is dedicated to one standard EMF implementation (EObjectImpl root class).
org.eclipse.gmt.modisco.java should rely on MinimalEObjectImpl since benchmarks show its interest as a default implementation. Some past wrong move/refactoring have been done between SVN repositories and MinimalEObjectImpl inheritance has been lost.

Comment 1 Nicolas Bros

2011-04-04 10:53:43 EDT

Fixed in revision 4233.

Comment 2 Fabien Giquel

2011-04-05 03:42:16 EDT

It is important to notice that modifying the inheritance does not violate API freeze by M6 :

- both MinimaEObjectImpl & EObjectImpl implement EObject interface
- EObjectImpl did not provide additional public/protected method compared to MinimaEObjectImpl.

Comment 3 Hugo Bruneliere

2011-08-18 07:52:37 EDT

Upgrade done.