[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-3093-g50a347f

simo idra at samba.org
Sun Jul 6 22:05:46 GMT 2008


On Sun, 2008-07-06 at 21:24 +0200, Volker Lendecke wrote:
> On Sun, Jul 06, 2008 at 03:05:36PM -0400, simo wrote:
> > Found the stuff really difficult to read otherwise.
> 
> Yes, I find some code quite hard to read as well, but that's
> the penalty for not reviewing code when it is checked in :-)
> 
> > Will keep the change local next time, btw if you want to review the
> > patch attached to bug 5571, that's why I started reformatting.
> 
> Why 
> 
> domain[PTR_DIFF(p, domuser)] = 0;
> 
> why not just
> 
> *p = 0;
> 
> This is just a stylistic question, I need to look at the
> logic of the code more. I've asked the reporter on irc to
> fill in more info how to reproduce the bug.

Dunno,
I guess I was looking at some similar code and just copied it over :)

can be replaced with *p = 0; of course.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Senior Software Engineer at Red Hat Inc. <ssorce at redhat.com>



More information about the samba-technical mailing list