RE: [Embedlets-dev] JAPL source code
Status: Alpha
Brought to you by:
tkosan
|
From: Christopher S. <cs...@oo...> - 2003-06-28 04:15:28
|
All good suggestions - I will make the changes. Chris > > Topic tags:[ARCH][JAPL][WIRING][DOCS][MGMT][STRATEGY][NEWBIE] > _______________________________________________ > > Chris wrote: > > > > > I would like to submit the following for your review: > > org > > |_japl > > |_ JaplListener.java > > |_ JaplEvent.java > > |_ network > > |_ MediaController.java > > |_ MediaEvent.java > > |_ MediaListener.java > > > > Hey, good start! A few of thoughts though... > > 1) I just looked through the standard library APIs that come with J2SE and > classes like SAXException, UIDefaults, URLDecoder, etc. and all keep the > letters of their acronyms in uppercase. If we follow this > convention (which I > would vote for) then JaplListener would be JAPLEvent, etc. > > 2) How about shortening 'network' to just 'net' and then putting > all of the > MediaController related classes inside a subpackage called > 'mediacontroller'? > For example: org.japl.net.mediacontroller.MediaController, > org.japl.net.mediacontroller.MediaEvent, > org.japl.net.mediacontroller.MediaListener. This way other > people can place > network related JAPL interfaces inside the 'net' package and they > won't all get > mixed up with other JAPL interface classes. > > 3) Should JAPLListener and JAPLEvent be placed inside of an > 'event' subpackage > or a 'util' subpackage similar to what is done in the standard > library APIs? > > > Ted > > __________________________________ > Do you Yahoo!? > SBC Yahoo! DSL - Now only $29.95 per month! > http://sbc.yahoo.com > > > ------------------------------------------------------- > This SF.Net email sponsored by: Free pre-built ASP.NET sites including > Data Reports, E-commerce, Portals, and Forums are available now. > Download today and enter to win an XBOX or Visual Studio .NET. > http://aspnet.click-url.com/go/psa00100006ave/direct;at.asp_061203_01/01 > _______________________________________________ > Embedlets-developer mailing list > Emb...@li... > https://lists.sourceforge.net/lists/listinfo/embedlets-developer > |