dirsync review
Matthieu Patou
mat at samba.org
Mon Mar 21 06:24:31 MDT 2011
On 21/03/2011 13:45, Matthias Dieter Wallnöfer wrote:
> Hi ekacnet,
>
> two comments from my side:
>
> - the indentation sometimes looks a bit strange - I cannot distinguish
> if this is only gitweb or if you have mixed tabs and whitespaces
Normally it should be tabs, I'll have a look if it's not linked with
code that I copy pasted from not so correctly formatted files !
> - with "int i" and "int k" in the dirsync implementation I'm still not
> very happy - I invite you to use the correct downto counting mechanism
> ("ldb_dn.c" shows how to do that in respect to the DN components)
Let's have a look not sure it could be solved as as you can notice we
are counting backward from num_values to 0 and if I use an unsigned int
when k = 0, the next k-- will give me 4294967295 and not -1, I can of
course cast k when doing a comparison not sure it's much more elegant.
Matthieu
--
Matthieu Patou
Samba Team http://samba.org
Private repo http://git.samba.org/?p=mat/samba.git;a=summary
More information about the samba-technical
mailing list