Draft #2: Re: [PATCH] SAMBA_3_0 : remove ldap_domain2hostlist()

[Love Hörnquist Åstrand]

"Gerald (Jerry) Carter" <jerry at samba.org> writes:

> Love wrote:
>
>>> Round #2.
>> 
>> You need to randomize the sorting for entries within same 
>> priority using weight so select one of the entires.
>
> Are you looking at the most recent code wqith the qsort() ?
> Granted I could randomize entries with equal weight and priority.
> But the current sort should be close.

I looked at the code with qsort, yes. 

If you have two entries, will both of the get selected with appropriate
distribution ? With your correct code, I don't think so.

priority=1,weight=2
priority=1,weight=1

Love

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 823 bytes
Desc: not available
Url : http://lists.samba.org/archive/samba-technical/attachments/20060505/45ba046f/attachment.bin