[PATCH] Add stackframes to public libsmbclient functions
simo
idra at samba.org
Mon Nov 19 16:35:27 GMT 2007
On Mon, 2007-11-19 at 15:37 +0100, Volker Lendecke wrote:
> On Mon, Nov 19, 2007 at 09:25:35AM -0500, Derrell Lipman wrote:
> > of the lower-level function that there's a problem down below. That's
> > not the way to do it IMO.
>
> Sad. Then we have to remove talloc_stackframe() completely
> again. I pretty much liked it, but as Simo has also already
> reject it, we probably need to remove them :-(
Volker, I am ok with stackframe
the only thing I don't digest yet is the use of an implicit destructor
(can be replaced in theory)
but samba3 code never uses destructors so I am ok with its use
> This will introduce tons of additional talloc_ctx arguments,
> in particular the DEBUG message arguments will become quite
> difficult to code. But going back to static pstrings is not
> an option in my opinion.
I think smaller patches will make it more digestable :-)
Simo.
--
Simo Sorce
Samba Team GPL Compliance Officer <simo at samba.org>
Senior Software Engineer at Red Hat Inc. <ssorce at redhat.com>
More information about the samba-technical
mailing list