[Monetdb-developers] Re: [Monetdb-checkins] MonetDB/src/modules/plain pcre.mx,1.14,1.15
Sjoerd Mullender wrote:
Update of /cvsroot/monetdb/MonetDB/src/modules/plain In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv22873
Modified Files: pcre.mx Log Message: Do not put anything above the license text (except possibly a "hash-bang" line (#!/...)). The script that updates the license text does not like this.
Furthermore, the standard in the MonetDB code is to use 8-space tabs. If you want to view things differently, that's fine, but make at least sure that it still looks correctly with 8-space tabs.
Since when? I have been told 4 width *tabs* (not spaces) is the standard in the MonetDB project. You seem to suggest here that I should use spaces instead... Off-topic: is there anyone who actually uses 8 width tabs within vi?
Fabian wrote:
Sjoerd Mullender wrote:
Update of /cvsroot/monetdb/MonetDB/src/modules/plain In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv22873
Modified Files: pcre.mx Log Message: Do not put anything above the license text (except possibly a "hash-bang" line (#!/...)). The script that updates the license text does not like this.
Furthermore, the standard in the MonetDB code is to use 8-space tabs. If you want to view things differently, that's fine, but make at least sure that it still looks correctly with 8-space tabs.
Since when? I have been told 4 width *tabs* (not spaces) is the standard in the MonetDB project. You seem to suggest here that I should use spaces instead...
I am not saying anything of the kind. Please use tabs (but not in Python source!). You are totally allowed to look at them as if they are 4 spaces wide. Just make sure that it *also looks correct* when they are viewed as 8 spaces wide. And don't put vim:... stuff at the top of files (I know, the one I removed wan't put there by you).
Off-topic: is there anyone who actually uses 8 width tabs within vi?
If I were to use vi, that's what I would use. -- Sjoerd Mullender
participants (2)
-
Fabian
-
Sjoerd Mullender