[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
RE: breaking changes in 4.0 for CDI (was: RE: [cdt-debug-dev] List of Plan items)
|
The instanceof should be used to check for IAdaptable anyway...
Speaking seriously, I would prefer to vote for a patch not for an idea.
I have the proposal document ready and I am going to add it to wiki.
-----Original Message-----
From: cdt-debug-dev-bounces@xxxxxxxxxxx
[mailto:cdt-debug-dev-bounces@xxxxxxxxxxx] On Behalf Of Alain Magloire
Sent: 23 October 2006 19:45
To: CDT Debug developers list
Subject: RE: breaking changes in 4.0 for CDI (was: RE: [cdt-debug-dev]
List of Plan items)
> From: cdt-debug-dev-bounces@xxxxxxxxxxx [mailto:cdt-debug-dev-
> bounces@xxxxxxxxxxx] On Behalf Of Mikhail Khodjaiants
> Sent: Wednesday, October 18, 2006 1:31 PM
> To: CDT Debug developers list
> Subject: RE: breaking changes in 4.0 for CDI (was: RE: [cdt-debug-dev]
> List of Plan items)
>
> > For example adding IAdaptable to ICDIObject and ICDISessionObject
> > are
> breaking changes.
>
> There are many ways to minimize the impact of this type of changes.
> For example, introduce new ICDIObject2 and ICDISessionObject2
> interfaces and add two new abstract classes:
>
> abstract class AbstractObject implements ICDIObject, ICDIObject2
>
> abstract class AbstractSessionObject implements ICDISessionObject,
> ICDISessionObject2
>
> Implementors can use theses classes as base classes for their
hierrachy.
> It's not a very good solution, but it covers most of the use cases.
>
> I agree, 4.0 is a major release, but we at least should try to make
> the transition easier.
>
It seems to be the same i.e. the client must still do instanceof:
if (obj1 instanceof ICDIObject2) {
ICDIObject2 obj2 = (ICDIObject2)obj1;
MyClass clz = obj2.getAdapter(MyClass.class); }
So still a lot of instanceof for little gain, IMHO.
So, why the hesitation? I would prefer to fix things right at the root
and a 4.0 release is a good candidate, IMHO?
Any other votes/takes on this?
_______________________________________________
cdt-debug-dev mailing list
cdt-debug-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-debug-dev
--
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
