Code review comment for lp:~jc.redoutey/libmemcached/mget_execute

Revision history for this message
Brian Aker (brianaker) wrote :

I am looking at porting this forward right now.

A couple of things:

1) When you modify memcached_st to add a new member, you need to make sure it is initialized correctly.
2) Mixing int with size_t will just cause porting issues (this was in function.c).

I need to give a better example of how to use just one server during a test.

Thanks for this work, I am sorry I didn't get it in faster.

Cheers,
   -Brian

« Back to merge proposal