Re: [TC6] Minimum coding style

2006-12-04 Thread Kevin Jackson
AS FOR STYLE, I THINK ALL UNNECESSARY WHITESPACE SHOULD BE BANNED AND TABS MADE ILLEGAL, THEY JUST INCREASE FILE SIZE TOO MUCH AND MY COMMODORE 64 JUST CAN'T HANDLE SOME OF THOSE LARGE SOURCE FILES ALL IN ONE GO. PLEASE WRITE YOUR JAVA LIKE THIS: public class Foobar{private String NAME;public

Re: [TC6] Minimum coding style

2006-11-29 Thread Jim Jagielski
On Nov 28, 2006, at 3:55 PM, Rainer Jung wrote: From a very personal view I'm tempted to share Mladens dislike of trailing spaces. If I try to express, why I don't like them, then it all comes back to diffs. Same here. Maybe 3/4 of a year ago I cleaned up the entire httpd codebase

Re: [TC6] Minimum coding style

2006-11-29 Thread Yoav Shapira
Hi, On 11/29/06, Jim Jagielski [EMAIL PROTECTED] wrote: Same here. Maybe 3/4 of a year ago I cleaned up the entire httpd codebase removing trailing spaces, wrong tabs, etc... It was painful, but it was worth it to occasionally do such housekeeping. But I haven't checked to see what, if

[TC6] Minimum coding style

2006-11-28 Thread Mladen Turk
Hi, Just committed two files, and again I've bumped upon the coding style. Although it seems we are quite good relating to the tabs, the trailing spaces are a real mess in some files. There are lots of files where we have if (...) { followed with the one or more spaces. Also there are lots empty

Re: [TC6] Minimum coding style

2006-11-28 Thread Remy Maucherat
Mladen Turk wrote: Hi, Just committed two files, and again I've bumped upon the coding style. Although it seems we are quite good relating to the tabs, the trailing spaces are a real mess in some files. There are lots of files where we have if (...) { followed with the one or more spaces. Also

Re: [TC6] Minimum coding style

2006-11-28 Thread Mladen Turk
Remy Maucherat wrote: Mladen Turk wrote: Comments? -1. This will screw up diffs, and these really irrelevant things will be added back very quickly. Tabs is one thing since it can be hard to work with, but this is simply ridiculous. Right, but at least the empty lines with spaces are

Re: [TC6] Minimum coding style

2006-11-28 Thread Remy Maucherat
Mladen Turk wrote: Also, I love the Filip style (sorry if that doesn't originates from you): for example, MessageDispatchInterceptor: }//end if }//sync }//end if Now, that is what I call ridiculous. Nice comment :( And what if he likes it and it makes his programming easier ?

Re: [TC6] Minimum coding style

2006-11-28 Thread Filip Hanik - Dev Lists
Mladen Turk wrote: Remy Maucherat wrote: Mladen Turk wrote: Comments? -1. This will screw up diffs, and these really irrelevant things will be added back very quickly. Tabs is one thing since it can be hard to work with, but this is simply ridiculous. Right, but at least the empty lines

Re: [TC6] Minimum coding style

2006-11-28 Thread Mladen Turk
Remy Maucherat wrote: -1 for enforcing any coding style (except tabs, since it can make code unreadable very easily). You must be joking, right? Regards, Mladen. - To unsubscribe, e-mail: [EMAIL PROTECTED] For

Re: [TC6] Minimum coding style

2006-11-28 Thread Jim Jagielski
On Nov 28, 2006, at 9:36 AM, Mladen Turk wrote: Remy Maucherat wrote: -1 for enforcing any coding style (except tabs, since it can make code unreadable very easily). You must be joking, right? One hopes... - To

Re: [TC6] Minimum coding style

2006-11-28 Thread Tim Funk
+1 trailing spaces always annoy me and text editors can easily be configured to strip them out. An alternative is to change the email diffs to diff ignoring whitespace changes. Then lines can have all the trailing spaces they want and we won't see those diffs. -Tim Mladen Turk wrote: Hi,

Re: [TC6] Minimum coding style

2006-11-28 Thread Remy Maucherat
Jim Jagielski wrote: On Nov 28, 2006, at 9:36 AM, Mladen Turk wrote: Remy Maucherat wrote: -1 for enforcing any coding style (except tabs, since it can make code unreadable very easily). You must be joking, right? One hopes... Why ? Because arbitrary rules only create tensions ?

Re: [TC6] Minimum coding style

2006-11-28 Thread Jim Jagielski
On Nov 28, 2006, at 10:44 AM, Remy Maucherat wrote: Jim Jagielski wrote: On Nov 28, 2006, at 9:36 AM, Mladen Turk wrote: Remy Maucherat wrote: -1 for enforcing any coding style (except tabs, since it can make code unreadable very easily). You must be joking, right? One hopes... Why

Re: [TC6] Minimum coding style

2006-11-28 Thread Remy Maucherat
Jim Jagielski wrote: I'm a firm believer in the One True Tab, but I lost that battle lng ago :) Yes, I used to push for one coding style a long time ago, but it's not going to happen (I figured it was better to collaborate with Costin rather than keep the clean - ahem - coding style and

Re: [TC6] Minimum coding style

2006-11-28 Thread Henri Gomez
I remember the cold days of the 'Tab brigade' :) 2006/11/28, Remy Maucherat [EMAIL PROTECTED]: Jim Jagielski wrote: I'm a firm believer in the One True Tab, but I lost that battle lng ago :) Yes, I used to push for one coding style a long time ago, but it's not going to happen (I figured

Re: [TC6] Minimum coding style

2006-11-28 Thread Mladen Turk
Henri Gomez wrote: I remember the cold days of the 'Tab brigade' :) Right. Anyhow, the point of my post was 'minimal', and removing the trailing spaces. It seems that the trailing spaces are very important, so fine with me. Cheers, Mladen.

Re: [TC6] Minimum coding style

2006-11-28 Thread Rainer Jung
From a very personal view I'm tempted to share Mladens dislike of trailing spaces. If I try to express, why I don't like them, then it all comes back to diffs. Trailing spaces increase the potential of diff lines with no functional differences. Nevertheless one wastes time to check very

Re: [TC6] Minimum coding style

2006-11-28 Thread Darryl Miles
Just in case people didn't know, diff does have a number of options to modify white space behavior, such that you can check in a massive whitespace cleanup patch but still view a clean diff which ignores blank lines, crlf/lf and leading/trailing whitespace, etc... So really its not that bad.

Re: [TC6] Minimum coding style

2006-11-28 Thread Kris Nuttycombe
Darryl Miles wrote: Just in case people didn't know, diff does have a number of options to modify white space behavior, such that you can check in a massive whitespace cleanup patch but still view a clean diff which ignores blank lines, crlf/lf and leading/trailing whitespace, etc... So really