Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev] Package visibility

+1 for making an exception.


On Wed, Jan 16, 2013 at 10:15 AM, Marc Khouzam <marc.khouzam@xxxxxxxxxxxx> wrote:

> -----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?

Doh!  When we created this package I didn't realize that we had
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

Back to the top