[PATCH] group_mapping: Avoid a talloc
Jeremy Allison
jra at samba.org
Wed Jan 8 14:35:22 MST 2014
On Thu, Jan 09, 2014 at 09:22:42AM +1300, Andrew Bartlett wrote:
>
> I can see the argument for stuff in tight inner loops, but I'm very,
> very wary of going back to fixed length strings everywhere. What's the
> whole argument of talloc_pool() that this was fast enough, without all
> the risks of fixed length strings?
>
> There is a great value in consistency of talloc-based string handling,
> because it means that we can realistically examine our fixed length
> string use, because it's rare, not used anywhere we might save a cycle
> or two outside the hot path.
>
> In short, I don't like where I see this headed, and I think we should
> consider that.
I can understand your reluctance on this. I also don't want
to see us going back to fixed-length buffers - that way
lies disaster :-).
But I think you need to trust Volker a little more on
this. Remember, this is something he's spend a *LOT* of
time looking at, and if he thinks we need a fixed length
value somewhere I'm inclined to give him the benefit of
the doubt.
In short, I think you raised a valid concern, but I think
it's time to let Volker push the code now :-).
Cheers,
Jeremy.
More information about the samba-technical
mailing list