I'll gladly sponsor this SRU for you ... but I'm a little bit concerned about the test case field being empty and I'm sure the SRU team (ppl who approved upload) will do as well.
While I can understand there is no clear reproducer for that bug ...
Could you please prove somehow that you did some dogfooding ?
Use libmemcached in a context as close as possible to the modified code ?
Ran a libmemcached/memcached test suite (if any) ?
Anything that can bring confidence in this patch. Especially that we have no upstream maintainer to review/accept it.
@joalif,
I'll gladly sponsor this SRU for you ... but I'm a little bit concerned about the test case field being empty and I'm sure the SRU team (ppl who approved upload) will do as well.
While I can understand there is no clear reproducer for that bug ... memcached test suite (if any) ?
Could you please prove somehow that you did some dogfooding ?
Use libmemcached in a context as close as possible to the modified code ?
Ran a libmemcached/
Anything that can bring confidence in this patch. Especially that we have no upstream maintainer to review/accept it.
Regards,
Eric