[PATCH] lib: Fix whitespace in params.c
Christopher R. Hertel
crh at samba.org
Mon Aug 11 06:31:11 MDT 2014
On 08/11/2014 05:47 AM, Volker Lendecke wrote:
> Hi!
>
> Review&push would be appreciated.
>
> Thanks,
>
> Volker
>
-1
If you are going to change the formatting style of a file, run it through
indent and change the whole thing to match the project "standard".
Also, this isn't a "fix", it's just a change. (This isn't a bug, it's a
preference.) The formatting style in this particular file is known as
"whitesmith", and it was once quite popular. I know, because it's my coding
style. I also know that I haven't touched that file since I rewrote it for
Samba 2 more than 10 years ago.
So, my vote is to either leave it as is or to completely convert it to the
current preferred formatting style.
Chris -)-----
--
"Implementing CIFS - the Common Internet FileSystem" ISBN: 013047116X
Samba Team -- http://www.samba.org/ -)----- Christopher R. Hertel
jCIFS Team -- http://jcifs.samba.org/ -)----- ubiqx development, uninq.
ubiqx Team -- http://www.ubiqx.org/ -)----- crh at ubiqx.mn.org
OnLineBook -- http://ubiqx.org/cifs/ -)----- crh at ubiqx.org
More information about the samba-technical
mailing list