portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Sean Taylor <da...@bluesunrise.com>
Subject Re: Small steps towards a long term goal (code quality)
Date Mon, 03 Mar 2003 17:14:05 GMT

On Monday, March 3, 2003, at 02:41  AM, Santiago Gala wrote:

> I am concerned about the overall state of interpackage dependencies in  
> Jetspeed. As I have been saying in different posts, if we don't have a  
> clear knowledge of the dependencies between our own packages and with  
> third party projects, it would be difficult to assess work needed for  
> PortletAPI support, release or maintain code, etc.
>
> Recently, Tom Copeland has published a report on "unused imports"  
> across jakarta projects in:
>
> http://nagoya.apache.org/eyebrowse/ 
> ReadMsg?listName=general@jakarta.apache.org&msgNo=14537
>
> Jetspeed is, currently, one of the projects with most unused imports.  
> I wonder if someone using an IDE to develop with Jetspeed could easily  
> send a patch which removes unused imports (without breaking the build,  
> if possible).
>
> I will eventually do it, but I need to install the proper tools I am  
> *very* scarce of time currently.
>
> This would be a very small step, but a path is always walked step by  
> step.
>
> The next one would be to use "deprecation=on" in the build, and ensure  
> that no deprecated methods are used, and then delete them, or at least  
> move them from "public" to protected or private.
>
> We have *way too many* public methods in Jetspeed, and this raises the  
> number of dependencies unnecesarily.
>
> I think we need to control more tightly the code base if we want to  
> succeed in the next months.
>
> Any thoughts?
>
Look into Maven.

Im not going to get involved in this one.
I've already starting cleaning up the code in the new code base, where  
you are welcome to help out.

--
David Sean Taylor
Bluesunrise Software
david@bluesunrise.com
+01 707 773-4646




---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jetspeed-dev-help@jakarta.apache.org


Mime
View raw message