On Thu, May 23, 2013 at 11:37:59AM +0800, Fam Zheng wrote:
> @@ -90,7 +98,16 @@ static int curl_aio_flush(void *opaque);
> static int curl_sock_cb(CURL *curl, curl_socket_t fd, int action,
> void *s, void *sp)
> {
> + BDRVCURLState *bs = s;bs is used for BlockDriverState in block/* code. I find this confusing. Usually the custom state is called 's'. Perhaps just rename the void* arguments s_ and sp_.
