svn commit: samba r16813 - in trunk/source: include nsswitch sam utils

simo idra at samba.org
Wed Jul 5 04:15:36 GMT 2006


On Wed, 2006-07-05 at 03:40 +0000, jra at samba.org wrote:
> Author: jra
> Date: 2006-07-05 03:40:40 +0000 (Wed, 05 Jul 2006)
> New Revision: 16813
> 
> WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=16813
> 
> Log:
> Rewrote much of the idmap code. This design is *shit*,
> and needs fixing. We must refactor out the cache code
> and separate it from the idmap_tdb so they are two
> separate pointer tables. They may call down into the
> same underlying functions but the MUST NOT BE ALLOWED
> to continue to mix up allocation and cachine. I'll know
> the design is correct when the flags field dissapears
> from the backend interface. Having said that this code
> is now a lot cleaner and easier to understand. Expect
> more hard words (and work) in this area.

Nice when you blame your own work this way :-)

> Anyone wanting to work on this code please see me
> first ! ("You behind the bike sheds, stand still
> laddie !" :-)

If you would just wait a second before rushing down this way ...

... on the 4th of July ...

I have no words ...

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org



More information about the samba-cvs mailing list