+1 --- Martin Cooper <[EMAIL PROTECTED]> wrote: > > > On Mon, 16 Dec 2002, Cedric Dumoulin wrote: > > > > > I said that I don't want to debate on this. I > know the reformated code > > doesn't follow the recomanded Jakarta rules code > standard. But another > > rule is to respect any other well formated > standard ;-). > > While I agree with respecting an original author's > choices, I don't agree > with having multiple sets of coding conventions in a > single code base. > That can only be confusing for everyone. In > addition, when we move to > Maven as the build system for Struts, we will have > to define a single > set of coding conventions for Checkstyle to work > with. I'm not suggesting > that we have to spend the time now to decide what > our conventions are, > just suggesting that we, at some point in time, need > to decide on a single > set of coding conventions for the entire Struts code > base. > > -- > Martin Cooper > > > > For me, the reformated code is really > unreadable: I can't detect the > > classes and methods structures at a glance, and so > it requires me some > > times to try to figure it out. I thing it is a > waste of time ... > > > > Cedric > > > > > > > > David Graham wrote: > > > > > With all due respect Cedric, that code did not > follow the java > > > standard coding guidelines so it was a candidate > for reformatting. > > > Under the Jakarta rules code must meet those > guidelines unless > > > specified differently for the project. AFAIK > Struts has no specific > > > rules so it defaults to the java standard. > > > > > > Following the java standard helps all developers > work on the project > > > faster and easier. > > > > > > Dave > > > > > > > > > > > > > > > > > > > > >> From: Cedric Dumoulin <[EMAIL PROTECTED]> > > >> Reply-To: "Struts Developers List" > <[EMAIL PROTECTED]> > > >> To: Struts Developers List > <[EMAIL PROTECTED]> > > >> Subject: Avoid code reformating ! > > >> Date: Mon, 16 Dec 2002 22:09:22 +0100 > > >> > > >> > > >> Hi everybody, > > >> > > >> Code formating is an ever ending debate. We, > as experienced > > >> developers, all have our preferences. When > there is several users as > > >> in Struts, there is a polite rule: do not > reformate code of others > > >> just because they don't follow your criteria. > > >> I have discover such reformating in code that > I have written. The > > >> problem is that I don't recognize the code > anymore, it takes me more > > >> time to do something, and comparisons tools > can't work anymore ;-( > > >> I don't want to debate about the "right" or > the "best" code > > >> formating. I just ask for the respect of each > others. > > >> So, for those having an automatic code > formatter, please disable it > > >> when playing with struts code, and resist to > the tentation ;-) > > >> > > >> Cedric > > >> > > >> > > >> -- > > >> To unsubscribe, e-mail: > > >> > <mailto:[EMAIL PROTECTED]> > > >> For additional commands, e-mail: > > >> <mailto:[EMAIL PROTECTED]> > > > > > > > > > > > > > _________________________________________________________________ > > > Add photos to your messages with MSN 8. Get 2 > months FREE*. > > > http://join.msn.com/?page=features/featuredemail > > > > > > > > > -- > > > To unsubscribe, e-mail: > > > > <mailto:[EMAIL PROTECTED]> > > > For additional commands, e-mail: > > > <mailto:[EMAIL PROTECTED]> > > > > > > > > > > > > -- > > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > > For additional commands, e-mail: > <mailto:[EMAIL PROTECTED]> > > > > > > > -- > To unsubscribe, e-mail: > <mailto:[EMAIL PROTECTED]> > For additional commands, e-mail: > <mailto:[EMAIL PROTECTED]> >
__________________________________________________ Do you Yahoo!? Yahoo! Mail Plus - Powerful. Affordable. Sign up now. http://mailplus.yahoo.com -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
