New sam system
Andrew Bartlett
abartlet at samba.org
Sat Aug 10 13:58:00 GMT 2002
Kai Krueger wrote:
>
> ----- Original Message -----
> From: "Andrew Bartlett" <abartlet at samba.org>
> Sent: Thursday, August 01, 2002 1:09 AM
>
> > Kai Krueger wrote:
> >
> > > P.S. perhaps adding const all along would make it clear which parameters
> are
> > > "in parameters" and which are "out paramters"
> >
> > Yes, that would be a very good idea.
>
> The appended patch adds const to all base function parameters that are "in"
> parameters. The parameters without const are therefore "out". The exception
> to this rule are the handles. In some functions the handles are "in"
> parameters and not changed to the callers point of view but still do not
> have const. This is because they might be changed internally (depending on
> the way those handles are implemented :-).
>
> In addition, I've started to add comments to all base functions. A lot of
> the function names are self explanatory, but I think some actually did
> profit of being explained :-)
Can you redo that patch so it applies to the copy in source/sam/ ?
Also, there are a few stray CRs in there...
Andrew Bartlett
--
Andrew Bartlett abartlet at pcug.org.au
Manager, Authentication Subsystems, Samba Team abartlet at samba.org
Student Network Administrator, Hawker College abartlet at hawkerc.net
http://samba.org http://build.samba.org http://hawkerc.net
More information about the samba-technical
mailing list