[PATCH] talloc: Tune talloc_vasprintf
Jeremy Allison
jra at samba.org
Thu May 15 02:23:07 MDT 2014
On Thu, May 15, 2014 at 10:19:53AM +0200, Volker Lendecke wrote:
> On Thu, May 15, 2014 at 10:17:02AM +0200, Stefan (metze) Metzmacher wrote:
> > Hi Volker,
> >
> > > A patch from my attic: In some profiles vsnprintf showed up
> > > high from talloc_asprintf. This cuts vsnprintf use in
> > > talloc_asprintf by half in most cases.
> > >
> > > Review would be appreciated!
> >
> > Do you think it would make sense to use PATH_MAX instead of 1024?
>
> Sure, we could do this. It's just that this routine has
> nothing to do with file names or paths. PATH_MAX might make
> people wonder why we refer to that in this routine.
>
> Any opinions?
I think PATH_MAX is too large. 1k is a reasonable
size - plus it's the size Volker profiled.
I tihnk it's good as-is.
More information about the samba-technical
mailing list