[nanomsg] Re: Problem with x64 build on windows
- From: Immanuel Weber <immanuel.weber@xxxxxxxxx>
- To: nanomsg@xxxxxxxxxxxxx
- Date: Thu, 26 Sep 2013 11:07:25 +0200
Hi Martin,
yesterday I allmost finished that modification. This morning a simpler
solution came to my mind: just include the efd.h header in device.c and
exchange the use of int for the file descriptors there from to nn_fd.
The patch for that is below, if you prefer your proposed solution, I can
provide a patch for that too. I thought about extending the use of nn_fd
to the nn_efd structs, but then again both nn_fd and nn_efd are plattform
depended types, so the use of the plattform native type is okay, I think.
Greetings
Immanuel
p.s. there are some compiler warnings about conversions scattered across
the project (regardless of the patch), where data loss can happen. Some of
them are quite severe, like size_t to uint8_t, which is on my machine a 8
to 1 byte conversion. I will try have a look at some of them.
The patch under MIT license (I definitely need to make the github thing
work):
Other related posts:
