You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geronimo.apache.org by Aaron Mulder <am...@alumni.princeton.edu> on 2006/05/04 08:05:22 UTC
Code Exchange
I was kind of inspired by Dain looking over the crufty
KernelManagementHelper (though unfortunately, I haven't yet finished
returning the favor).
In any case, I could produce a list of a few classes that are big and
ugly and need refactoring. For instance, there are a couple rewritten
yet still monolithic portlets that should be broken up into separate
handlers for separate screens, and the plugin installer could use
review and splitting out of logic like the HTTP connections and
probably more...
If anyone wants to review and refactor some of these in the post-1.1
timeframe, please let me know -- I wouldn't mind trading. :)
Thanks,
Aaron
Re: Code Exchange
Posted by Aaron Mulder <am...@alumni.princeton.edu>.
Nice. :)
Aaron
On 5/4/06, David Blevins <da...@visi.com> wrote:
> On May 3, 2006, at 11:05 PM, Aaron Mulder wrote:
>
> > I was kind of inspired by Dain looking over the crufty
> > KernelManagementHelper (though unfortunately, I haven't yet finished
> > returning the favor).
> >
> > In any case, I could produce a list of a few classes that are big and
> > ugly and need refactoring. For instance, there are a couple rewritten
> > yet still monolithic portlets that should be broken up into separate
> > handlers for separate screens, and the plugin installer could use
> > review and splitting out of logic like the HTTP connections and
> > probably more...
> >
> > If anyone wants to review and refactor some of these in the post-1.1
> > timeframe, please let me know -- I wouldn't mind trading. :)
> >
>
> On those notes, I've got a slightly cleaned up version of your
> original DirectoryMonitor i've been meaning to put back into the
> mainline. Was fun picking it down to it's minimum without loosing
> any functionality.
>
> -David
>
>
Re: Code Exchange
Posted by David Blevins <da...@visi.com>.
On May 3, 2006, at 11:05 PM, Aaron Mulder wrote:
> I was kind of inspired by Dain looking over the crufty
> KernelManagementHelper (though unfortunately, I haven't yet finished
> returning the favor).
>
> In any case, I could produce a list of a few classes that are big and
> ugly and need refactoring. For instance, there are a couple rewritten
> yet still monolithic portlets that should be broken up into separate
> handlers for separate screens, and the plugin installer could use
> review and splitting out of logic like the HTTP connections and
> probably more...
>
> If anyone wants to review and refactor some of these in the post-1.1
> timeframe, please let me know -- I wouldn't mind trading. :)
>
On those notes, I've got a slightly cleaned up version of your
original DirectoryMonitor i've been meaning to put back into the
mainline. Was fun picking it down to it's minimum without loosing
any functionality.
-David