Update ldb-perfs branch
simo at samba.org
Sun Jun 7 14:59:53 MDT 2015
On Sat, 2015-06-06 at 22:39 -0700, Matthieu Patou wrote:
> Hello All,
> I rebased and updated my branch related to ldb-perfs it's at
> It pass make test on sn-devel.
the code looks mostly good but I am looking at ldn_dn_set_cadefolded()
and not really liking it. That API makes it way too easy to do great
Given you really need it only in ltdb_index_filter() to fetch a DN entry
knowing it was stored casefolded, I would rather introduce a function
like: ldb_dn_from_casefolded_ldb_val() (or even
ldb_dn_from_index_ldb_val) The implementation would share most of the
code with the original ldb_dn_from_ldb_val() and just add the casefolded
dn in the proper internal structure.
This way we avoid the risk of someone abusing ldn_dn_set_cadefolded() in
places where it shouldn't be.
More information about the samba-technical