[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
RE: [dsdp-tm-dev] Request for Cleanup Change on ISystemMessages
|
Hello,
FYI: I made the change on ISystemMessages as proposed.
As a result, the compressed (!) rse.doc.isv plugin
is now 370 Kbytes smaller -- 10.7MB instead of 11.4.
I suggest you update your workspace to get all the changes.
Cheers,
--
Martin Oberhuber
Wind River Systems, Inc.
Target Management Project Lead, DSDP PMC Member
http://www.eclipse.org/dsdp/tm
> -----Original Message-----
> From: dsdp-tm-dev-bounces@xxxxxxxxxxx
> [mailto:dsdp-tm-dev-bounces@xxxxxxxxxxx] On Behalf Of
> Oberhuber, Martin
> Sent: Thursday, August 24, 2006 8:02 PM
> To: David Dykstal; David McKnight; Kushal Munir
> Cc: Target Management developer discussions
> Subject: [dsdp-tm-dev] Request for Cleanup Change on ISystemMessages
>
> Hi Committers,
>
> There are some interfaces in RSE which just define constants:
>
> ISystemMessages
> ISystemIconConstants
>
> Especially for ISystemMessages, many RSE classes _implement_
> the constants "pseudo-interface" in order to get access to
> the constants without having to qualify them.
>
> I consider this abuse of a Java feature, since those classes
> do not really "implement" any messages -- it's more of an
> implementation detail what messages they choose to use.
> In addition to this abuse, the current code also has one very
> negative effect: It blows up our Javadoc unnecessarily, because
> each and every class that implements ISystemMessages has a
> large amount of "fields inherited from base classes". See
>
> http://dsdp.eclipse.org/help/latest/topic/org.eclipse.rse.doc.
> isv/refere
> nce/api/org/eclipse/rse/ui/ISystemMessages.html
>
> http://dsdp.eclipse.org/help/latest/topic/org.eclipse.rse.doc.
> isv/refere
> nce/api/org/eclipse/rse/ui/wizards/SystemNewConnectionWizardDe
> faultMainP
> age.html
>
> I would like to refactor the code, such that all access to
> Ids in ISystemMessages is qualified and the classes do not
> implement the "pseudo-interface".
>
> Comments?
> Votes?
>
> Thanks,
> --
> Martin Oberhuber
> Wind River Systems, Inc.
> Target Management Project Lead, DSDP PMC Member
> http://www.eclipse.org/dsdp/tm
> _______________________________________________
> dsdp-tm-dev mailing list
> dsdp-tm-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/dsdp-tm-dev
>