tabs and formating - was Re: [PATCH] winbind id assignment module
John E. Malmberg
wb8tyw at qsl.net
Fri May 24 08:07:03 GMT 2002
Mike Gerdts wrote:
> On Sat, 2002-05-18 at 20:54, Andrew Bartlett wrote:
>
>>This looks *much* better.
>>
>>I'm not sure on the 'reload' functionality, but I suppose its a good
>>idea. Other than minor things like indenting, (Try 8-space tabs) I
>>think this is well on its way to inclusion.
>
> As for spacing... I tried to follow the standard that I saw in the file
> already. I used 4 character tabs, but they should have expanded out
> OK. I thought that others were using 4 character tabs as well because
> sections of winbindd_idmap.c have tabs expanded to four characters. In
> any case, the next version that goes out will get rid of any expanded
> tabs and any necessary reformatting for prettiness will take place.
Almost all terminal emulators and printing devices assume that the tab
character stops on 8 character boundaries.
Some editors will locally override it.
So be aware that if you have set your tab stops to 4 characters, it will
probably show up as 8 characters on many systems.
Using all spaces instead of tabs solves that problem, but it really is
convenient to have tabs there when using the arrow keys to navigate in a
file.
There does not seem to be a style book for coding on the SAMBA project.
-John
wb8tyw at qsl.network
Personal Opinion Only
More information about the samba-technical
mailing list