s4:ldb_tdb - Rework/Various

[Andrew Bartlett]

On Mon, 2009-10-12 at 15:51 +0200, Matthias Dieter Wallnöfer wrote:
> Hi Andrew,
> 
> well I don't like GOTOs as well. But I took a very diffused practice in 
> s4 code (result/error handling goto). If you are very unhappy with the 
> changed code I'll add additional cosmetics through a new patch which 
> revert them partially.

For the cases where there really is no cleanup, I would like them
reverted.  

For the cases where there is a trivial cleanup, perhaps talk to simo
about how he would prefer it (he is the ldb maintainer after all), but
my preference is to have a duplicated 'talloc_free(foo); return ret;'
instead of a goto. 

Andrew Bartlett
-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Samba Developer, Cisco Inc.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20091013/a1b38e44/attachment.pgp>