[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] curl: Cast fd to int for DPRINTF
From: |
Fam Zheng |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH] curl: Cast fd to int for DPRINTF |
Date: |
Tue, 2 Aug 2016 09:09:06 +0800 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Mon, 08/01 12:41, John Snow wrote:
> > - DPRINTF("CURL (AIO): Sock action %d on fd %d\n", action, fd);
> > + DPRINTF("CURL (AIO): Sock action %d on fd %d\n", action, (int)fd);
> > switch (action) {
> > case CURL_POLL_IN:
> > aio_set_fd_handler(s->aio_context, fd, false,
> >
>
> Is curl_socket_t always of type long long unsigned int? why not use %llu ?
Since it is a typedef, variation between platforms is imaginable. On Linux it
is 4 bytes, the error is only seen on Windows.
Fam