From: Erik V. <eri...@xs...> - 2011-10-07 14:03:05
|
In my previous mail I mentioned a Perl XML formatter that I have just written on the fly and committed to the repository (well, I had written such a script before, but could not find it). I have used it to format some recreated Tiles.xml files, and will keep using it for that purpose (perhaps for other XML files too, if needed; but XMLBuddy seems to work the same). On the details, I have settled on the format of the generic Tiles.xml file, as it has recently been reformatted by Brett: - for indentation: tabs, no spaces (I loathe tabs, but see no point in starting an argument about that), - newlines only, no CRs (to achieve that on Windows, formatxml.pl uses binmode). Can we try to standardize this way? I often see redundant whitespace-only differences in commits, which is a waste. Redundant whitespace-only differences are also often seen in Java source code commits. Here I would also propose to standardize on the currently most frequent usage, which is: - for indentation: spaces, no tabs (after all, in this way I have written the majority of the code, and Stefan does it similarly. I notice that Brett uses tabs). - newlines only (I don't think that it matters for Java compilers, but this is how Eclipse is doing it even on my Windows machine. Brett's machine is adding CRs, as it seems to me). Please note, that my Eclipse is set to convert all tabs to spaces on saving Rails Java files. This all is no problem as long as we work on different code packages, but some files have several contributors by now. I hope this helps. Erik. |
From: brett l. <bre...@gm...> - 2011-10-07 14:21:27
|
On Fri, Oct 7, 2011 at 10:02 AM, Erik Vos <eri...@xs...> wrote: > In my previous mail I mentioned a Perl XML formatter that I have just > written on the fly and committed to the repository (well, I had written such > a script before, but could not find it). I have used it to format some > recreated Tiles.xml files, and will keep using it for that purpose (perhaps > for other XML files too, if needed; but XMLBuddy seems to work the same). > > On the details, I have settled on the format of the generic Tiles.xml file, > as it has recently been reformatted by Brett: > - for indentation: tabs, no spaces (I loathe tabs, but see no point in > starting an argument about that), > - newlines only, no CRs (to achieve that on Windows, formatxml.pl uses > binmode). > Can we try to standardize this way? I often see redundant whitespace-only > differences in commits, which is a waste. > > Redundant whitespace-only differences are also often seen in Java source > code commits. Here I would also propose to standardize on the currently most > frequent usage, which is: > - for indentation: spaces, no tabs (after all, in this way I have written > the majority of the code, and Stefan does it similarly. I notice that Brett > uses tabs). > - newlines only (I don't think that it matters for Java compilers, but this > is how Eclipse is doing it even on my Windows machine. Brett's machine is > adding CRs, as it seems to me). > Please note, that my Eclipse is set to convert all tabs to spaces on saving > Rails Java files. > > This all is no problem as long as we work on different code packages, but > some files have several contributors by now. > > I hope this helps. > > Erik. > In recent years, I've been using Windows, Mac, and Linux machines to commit code. So, my formatting rules have been pretty inconsistent. Apologies for my sloppiness. I prefer spaces to tabs. I also prefer CR to CR/LF (I blame Windows for the CR/LFs). So, no arguments with any of that. :-) ---Brett. |
From: Stefan F. <ste...@we...> - 2011-10-09 15:44:03
|
In a standard setup of Eclipse the formats for .java files should be set correctly from Rails-format.xml automatically. For .xml I checked the setup the preferences of the Rhino add-ons manually again and it is set to tabs. I have no specific preference for spaces/tabs (but if I am forced to choose I prefer tabs), however I strongly prefer if adherence to a decision ;-) Stefan On Friday, October 07, 2011 04:02:59 pm Erik Vos wrote: > In my previous mail I mentioned a Perl XML formatter that I have just > written on the fly and committed to the repository (well, I had written > such a script before, but could not find it). I have used it to format > some recreated Tiles.xml files, and will keep using it for that purpose > (perhaps for other XML files too, if needed; but XMLBuddy seems to work > the same). > > On the details, I have settled on the format of the generic Tiles.xml file, > as it has recently been reformatted by Brett: > - for indentation: tabs, no spaces (I loathe tabs, but see no point in > starting an argument about that), > - newlines only, no CRs (to achieve that on Windows, formatxml.pl uses > binmode). > Can we try to standardize this way? I often see redundant whitespace-only > differences in commits, which is a waste. > > Redundant whitespace-only differences are also often seen in Java source > code commits. Here I would also propose to standardize on the currently > most frequent usage, which is: > - for indentation: spaces, no tabs (after all, in this way I have written > the majority of the code, and Stefan does it similarly. I notice that > Brett uses tabs). > - newlines only (I don't think that it matters for Java compilers, but this > is how Eclipse is doing it even on my Windows machine. Brett's machine is > adding CRs, as it seems to me). > Please note, that my Eclipse is set to convert all tabs to spaces on saving > Rails Java files. > > This all is no problem as long as we work on different code packages, but > some files have several contributors by now. > > I hope this helps. > > Erik. > > > > > > --------------------------------------------------------------------------- > --- All of the data generated in your IT infrastructure is seriously > valuable. Why? It contains a definitive record of application performance, > security threats, fraudulent activity, and more. Splunk takes this data > and makes sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2dcopy2 > _______________________________________________ > Rails-devel mailing list > Rai...@li... > https://lists.sourceforge.net/lists/listinfo/rails-devel |