[PATCHES] Various cleanups and minor restructuring
Martin Schwenke
martin at meltin.net
Fri Sep 19 21:48:46 MDT 2014
On Fri, 19 Sep 2014 17:04:13 +0200, Volker Lendecke
<Volker.Lendecke at SerNet.DE> wrote:
> Pushed most of them.
Thanks.
> I did not push the move of mkproto. I think we should not
> depend on that for common code. I know we're not there, but
> putting mkproto into common gives a bad signal.
OK, I agree. While I'm fixing up lib/util includes, I'll actually a
static unix_privs.h that does the right stuff... although unix_privs.h
isn't a public header.
> For the talloc_tos removal I've got a different proposal.
> See attached. If you prefer yours, you can for sure push it
> with my R-b. It is certainly a correct patch.
Pushed yours. I had asked Amitay to double-check that talloc_tos() was
unnecessary and could be replaced by NULL, given that I thought I
understood talloc_tos(), but wasn't 100% sure I hadn't missed something
subtle. He said, "yeah, either that or a static array of size
PATH_MAX". I opted for the simpler patch... but 1 less talloc() is
good. :-)
More soon...
peace & happiness,
martin
More information about the samba-technical
mailing list