[PATCH] More tdb/ldb tests.
Adrian Cochrane
adrianc at catalyst.net.nz
Tue Sep 1 03:01:53 UTC 2015
Sorry about this slodge of eMails but I forgot to attach that latest
patch.
On Tue, Sep 1, 2015 at 2:59 PM, Adrian Cochrane
<adrianc at catalyst.net.nz> wrote:
> And I caught a bug in it (which means for example "cnNotHere" matches
> the "cn" attribute). This latest modification to your patch fixes it.
> Once again I'm looking for your signed off by to be added to it.
>
> Thanks
>
> On Tue, Sep 1, 2015 at 12:08 PM, Adrian Cochrane
> <adrianc at catalyst.net.nz> wrote:
>> I've updated the patch I wanted you to sign to some newer feedback.
>> Can I please have your signed off by on this newer modification
>> please?
>>
>> On Wed, Aug 26, 2015 at 3:52 PM, Adrian Cochrane
>> <adrianc at catalyst.net.nz> wrote:
>>> Dear Matthieu,
>>> I've modified your patch per Volker's comments, can I please have
>>> your signed off by on the modified patch so we can merge this and
>>> the tests I've added into master.
>>>
>>> Thanks,
>>> Adrian Cochrane
>>>
>>> On Wed, Aug 26, 2015 at 3:17 PM, Adrian Cochrane
>>> <adrianc at catalyst.net.nz> wrote:
>>>> Caught a slight oversight in what I just posted.
>>>>
>>>> On Tue, Aug 25, 2015 at 11:01 AM, Adrian Cochrane
>>>> <adrianc at catalyst.net.nz> wrote:
>>>>> I've now tidied up Matthieu Patou's patches.
>>>>>
>>>>> On Fri, Aug 21, 2015 at 6:05 PM, Volker Lendecke
>>>>> <Volker.Lendecke at SerNet.DE> wrote:
>>>>>> On Fri, Aug 21, 2015 at 01:49:28PM +1200, Andrew Bartlett wrote:
>>>>>>> On Fri, 2015-08-21 at 13:27 +1200, Adrian Cochrane wrote:
>>>>>>> > I think I addressed all your issues with this code.
>>>>>>>
>>>>>>> Thanks.
>>>>>>>
>>>>>>> Reviewed-by: Andrew Bartlett <abartlet at samba.org>
>>>>>>>
>>>>>>> Can I get a second team reviewer please?
>>>>>>
>>>>>> Two comments: First, in ldb_unpack_data_withlist we can live
>>>>>> without the "filter" boolean. If you want to protect against
>>>>>> "list==NULL" but "list_size!=0" just do a
>>>>>>
>>>>>> if (list == NULL) { list_size = 0; }
>>>>>>
>>>>>> Second, parts of the patch go way beyond the 80-char limit.
>>>>>> I think introducing a helper variable might be overdue here.
>>>>>>
>>>>>> Volker
>>>>>>
>>>>>> --
>>>>>> SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
>>>>>> phone: +49-551-370000-0, fax: +49-551-370000-9
>>>>>> AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
>>>>>> http://www.sernet.de, mailto:kontakt at sernet.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-ldb-introduce-ldb_unpack_data_withlist-to-unpack-par.patch
Type: text/x-patch
Size: 7185 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20150901/a168a61f/0003-ldb-introduce-ldb_unpack_data_withlist-to-unpack-par.bin>
More information about the samba-technical
mailing list