[PATCH] Second chunk of parameter internal variable renames

Jeremy Allison jra at samba.org
Fri Jan 24 16:14:55 MST 2014


On Fri, Jan 24, 2014 at 10:09:30PM +1300, Andrew Bartlett wrote:
> Alexander or Christian (or anyone else on the team interested),
> 
> Could you possibly review this second chunk of internal loadparm rename
> patches for me?
> 
> These patches are part of our branch at
> git://git.catalyst.net.nz/samba.git polished-param2

Hi Andrew,

I really tried hard to review+push these for you
('cos the look completely boilerplate and safe
to me) but unfortunately this first one depends
on a missing patch - the patch matching algorithm
fails in lib/param/param_functions.c as the
FN_LOCAL_XXXX text doesn't match on the first
change. The second and subsequent patchsets
then have the same problem :-(.

Can you split out all the changes to the
CamelCase -> camel_case parameters into
one (large) set of patches patchset and
I'll review them all in one go. I think
that's the easiest way to stop running
into silly gotcha's like this and get
the rename in.

Cheers,

	Jeremy.


More information about the samba-technical mailing list