JSLFrame and JSLDeskTop (Re: [Jsynthlib-devel] Bug report: Actions.saveFrame())

[Hiroo H. <hir...@co...>]

Hi Rib,

I've checked in JSLFrame and JSLDeskTop which have javadoc on public
methods.  I just borrowed the javadoc comment from JInternalFrame and
JDesktopPane.  When you have time, please add more comments to help
understand the code.

And I found the most of JSynthLib dependent code can be removed from
these classes by adding some arguments on the constructor of JSLDeskTop.=20
If you don't mind,  I'll make some changes.

Rib> That sounds right, but I don't have time right now to check for sure.=
=20
Rib> I'll try to add some documentation as soon as I have a chance.
=2E..
Rib> > BTW could you add some comments on your JSL* classes?  Now they have
Rib> > almost no comments.  It was difficult for me to understand.  And you=
 (we)
Rib> > are thinking to move them into a independent package to be shared wi=
th
Rib> > other application.  They needs some Java comments to be reused.
Rib> >=20
Rib> > And they has some code depending for JSynthLib, AppConfig.getGuiStyl=
e(),
Rib> > PatchEdit.exit(), etc.  Some hooks will be defined to support them f=
rom
Rib> > outside of the package.
--=20
Hiroo Hayashi