Ok, I'm failing to stay out of this argument once again. Crap.
On Mon, Jun 18, 2001 at 08:36:52AM -0700, [EMAIL PROTECTED] wrote:
> /*
> - * JBoss, the OpenSource EJB server
> - *
> - * Distributable under LGPL license.
> - * See terms of license at gnu.org.
> - */
> +* JBoss, the OpenSource EJB server
> +*
> +* Distributable under LGPL license.
> +* See terms of license at gnu.org.
> +*/
Huh? Why?
> package x;
>
> +//EXPLICIT IMPORTS
> +import a.b.C1; // GOOD
> +import a.b.C2;
> +import a.b.C3;
>
> +// DO NOT WRITE
> +import a.b.*; // BAD
No argument on this.
> +/**
> +* <description>
> +*
> +* @see <related>
> +* @author <a href="mailto:{email}";>{full name}</a>.
> +* @author <a href="mailto:[EMAIL PROTECTED]";>Marc Fleury</a>
> +* @version $Revision: 1.4 $
> +*
> +* Revisions:
> +*
> +* yyyymmdd author: explicit fix description (no line numbers but methods) go
>beyond the cvs commit message
> +* eg:
> +* 20010516 marc fleury: Ask all developers to clearly document the Revision,
>changed the header.
I'm not going to repeat the argument we had a while back about this, but I
still believe that this is a Very Bad Idea. It will not be maintained because
it's duplication of work that people already do when they write CVS commit
logs.
Here's a solution to your perceived problem of not having access to the CVS
history when offline: maintain a local mirror of the CVS repository on
your machine.
Also, if you are going to do this, put some HTML in there so that the
javadocs are readable.
> +// DO NOT USE "TAB" TO INDENT CODE USE *2* SPACES FOR PORTABILITY AMONG EDITORS
Ok, fine. Why *2* spaces all of a sudden? AFAIK, all of the existing code
is 3 or 4 spaces. In some ways, TABs are better for editor portability
because people can choose the width of the TABs that they prefer. I prefer
spaces, however, so I'm not going to argue.
It seems to me that if the goal is to make it as easy as possible for people
to contribute to JBoss then there should be an absolute minimum of red tape
and bizzarre coding standards.
Toby.
_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development
