Eclipse Community Forums: Sapphire » internal packages in sapphire.ui

Home » Archived » Sapphire » internal packages in sapphire.ui

internal packages in sapphire.ui [message #1144196]

Fri, 18 October 2013 14:16

Eclipse User

Hey Konstantin,

I noticed during recent refactoring you have several new internal packages in the org.eclipse.sapphire.ui bundle, but those are not exported in the Manifest. I know that its a no-no but right now in my adopter product I'm having to just straight copy some of those classes and I'd rather just reference them directly (e.g. RadioButtonsGroup)

Would you be willing to add those internal packages to exported packages in manifest for us adopters who don't mind the risk of using internal classes? Smile

I could discuss the reasons for needing some internal packages, but I feel that my use-case is esoteric enough it may not be that useful to be added to the framework itself.

[Updated on: Fri, 18 October 2013 14:17] by Moderator

Re: internal packages in sapphire.ui [message #1144226 is a reply to message #1144196]

Fri, 18 October 2013 14:45

Eclipse User

Why don't you go ahead and raise an enhancement request to make classes you are using public? Probably one bugzilla per class unless the usecase for several classes is somehow tightly coupled. I am not opposed to making more classes public, but I don't like exporting internal packages. I find that it allows for sloppy coding where developers reference internal classes when a perfectly good public class is available. Easy access to internals also makes it easy to forget to file requests to make something public, so over time the project looses ability to know the scope of the real API necessary by adopters.

[Updated on: Fri, 18 October 2013 14:53] by Moderator

Re: internal packages in sapphire.ui [message #1144253 is a reply to message #1144226]

Fri, 18 October 2013 15:10

Eclipse User

Sure thing. Here is the first one: https://bugs.eclipse.org/bugs/show_bug.cgi?id=419887

Previous Topic:	If/Then/Else no longer available in sdef?
Next Topic:	Runtime Exception when opening Editor

Goto Forum:

-=] Back to Top [=-

Current Time: Sun Jul 27 14:52:49 EDT 2025

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter