[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [cdt-dev] Generics and APIs
|
Does it mean that you agree on changing APIs to use generics? I think it
is a separate topic.
Schaefer, Doug wrote:
>
> Too late. We've decided on 1.5.
>
> Doug.
>
> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx]
> On Behalf Of Elena Laskavaia
> Sent: Thursday, April 10, 2008 4:05 PM
> To: CDT General developers list.
> Subject: Re: [cdt-dev] Generics and APIs
>
> Personally I think we should avoid java 1.5 all together, and I don't
> think we should change API just to remove warnings. I think these
> particular warnings should be turned off.
>
> Jason Montojo wrote:
> >
> > Hi everyone,
> >
> > I've been fixing compiler warnings about generics in the language
> > mapping classes. This involved having to change a number of method
> > signatures to account for parameterized collections. For example, I
> > changed this in
> > LanguageManager:
> >
> > Old:
> > public Map getContentTypeIdToLanguageDescriptionsMap(){ ... }
> >
> > New:
> > public Map<String, ILanguageDescriptor[]>
> > getContentTypeIdToLanguageDescriptionsMap(){ ... }
> >
> > Such a change wouldn't break existing clients and it makes the API
> > much easier to work with. Regardless, it is still an API change.
> >
> > Should I go ahead and make the changes to the signatures? Or should I
>
> > leave them as-is? I know we're already past the API freeze so I'd
> > like to know how the community feels about this.
> >
> > Thanks,
> > Jason Montojo
> > IBM CDT Team
> > IBM Toronto Lab
> > 905-413-5228
> > jmontojo@xxxxxxxxxx
> >
> > _______________________________________________
> > cdt-dev mailing list
> > cdt-dev@xxxxxxxxxxx
> > https://dev.eclipse.org/mailman/listinfo/cdt-dev
> >
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>