| Re: [cdt-dev] Package visibility |
Doh! When we created this package I didn't realize that we had
> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx
> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Marc Dumais
> Sent: Wednesday, January 16, 2013 1:11 PM
> To: cdt-dev@xxxxxxxxxxx
> Subject: [cdt-dev] Package visibility
>
> Hi,
>
> While working on adding CPU load information to the multicore
> visualizer
> (396268), I had to add a method to the interface
> org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils.DSF
> DebugModelListener.
>
>
> When I did, I got an error in Eclipse that the interface is
> an API that
> intends to be implemented. Looking in the plugin.xml, it seems that
> even if the package
> "org.eclipse.cdt.dsf.gdb.multicorevisualizer.internal.utils" has
> "internal" in its name, the package visibility doesn't
> reflect that. So
> technically I am breaking the API by adding a method to
> DSFDebugModelListener.
>
> Any ideas/suggestions on how best to proceed?
to set the visibility to hidden. I just thought using the string
"internal" was the trigger for making the package internal.
This was meant to be internal from the start. Are we stuck
with it as API, or can we get away with marking it hidden now?
I don't expect anyone to be using this, but officially it would
break the API.
Any opinions for CDT 8.2? Can we make an exception in the API
tooling or do we have to live with the mistake?
I vote for making an exception, but I may be biased ;)
_______________________________________________
cdt-dev mailing list
cdt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/cdt-dev