----- Original Message -----
From: "Daniel Rall" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Wednesday, June 20, 2001 11:00 AM
Subject: Re: Package cleanup proposal


> Jason van Zyl <[EMAIL PROTECTED]> writes:
>
> > On 6/15/01 4:47 PM, "Kasper Nielsen" <[EMAIL PROTECTED]> wrote:
> > > 6.
> > > move org.apache.turbine.util.migrator ->
> > > org.apache.turbine.tdk.util.migrator (and keep it in the TDK
repository).>
> >
> > +1
>
> +1
>
> > > 7.
> > > Do something about org.apache.turbine.util.Fileutils (What's it for?)
> >
> > I'm not sure where it's used.
>
> It's used in Tigris.  I can remove it if desired; to me it makes the
> most sense in Turbine, though.
>
> > > 8.
> > > move SequencedHashTable & StingStackBuffer to Commons->collections
> >
> > +1
>
> +1, provided that I have a tagged and and released version of the
> Commons Collections to use (I've been too busy to track progress
> lately and don't know what's going on with that).
>
> > > 9.
> > > Find somewhere for commom Turbine App classes like
> > > DateSelector.java
> > > TimeSelector.java
> > > Selectorbox.java
> > > perhaps a new subproject? (org.apache.turbine.windmill ?)
> >
> > These should probably be replaced by context tools, or
> > velocity #macros.
>
> -0
>
> > > 10.
> > > Be a little more critical about what util classes that ends up in
> > > org.apache.turbine.util in the future.
>
> What critieria do you suggest?
>
I just want people thinking about perhaps putting classes in Commons, thats
where everybody can benefit for them, and we have one less class to worry
about.
I really think that the number of classes that a project has in propotional
to amount of time you have use to get a good overview of the application.
Allmost everybody is overwhelmed with the size of turbine allready, off
course this should all change to the better when we get all the refactoring
done.

- Kasper


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to