[PATCH] Add stackframes to public libsmbclient functions
ab at samba.org
Tue Nov 20 16:16:50 GMT 2007
Volker Lendecke пишет:
> On Tue, Nov 20, 2007 at 09:55:31AM +0300, Alexander Bokovoy wrote:
>>> 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
>> DBWRAP code in Samba 3 uses destructors, see
>> source/lib/dbwrap_tdb.c:db_tdb_record_destr(). It even allocates
>> childs in destructor in relies on 'childs freed after calling
>> destructor' semantics.
> If all of you are against this construct, why can't you comment on it
> when I checked it in in the first place? Now it is going to be a
> fucking hell lot of work to remove it all again!!
Now you put me in defending position. :-) I like this trick though I
made additional comments to it in my ctdb-merge branch after checking
through talloc code to make no surprises for anyone else in future.
Nothing against you and the code (who we are to be against? Code speaks!).
/ Alexander Bokovoy
Samba Team http://www.samba.org/
ALT Linux Team http://www.altlinux.org/
Midgard Project Ry http://www.midgard-project.org/
More information about the samba-technical