> > Would there be interest in adding a function that returned
> > this information?
> >
> > int zmq_socket_type(void* socket);
> 
> Yep, this look dead on.  I would love to see this.

I propose the attached patch, which I submit licensed under MIT/X11.

(It is two patches because the first one changes by mistake the modes
for several source files. Stupid Windows.)

One comment: I get five warnings under Visual C++ 2008, all like this:

zeromq2\src\pull.cpp(27) : warning C4355:
  'this' : used in base member initializer list

(pull.cpp, push.cpp, sub.cpp and xreq.cpp twice). They look of no
concern, but there you have them.

-- 
Gonzalo Diethelm

Attachment: 0002-Fixed-mode-of-files-changed-by-my-last-commit.-Stupi.patch
Description: 0002-Fixed-mode-of-files-changed-by-my-last-commit.-Stupi.patch

Attachment: 0001-Keep-the-socket-type-in-a-member-of-socket_base_t-an.patch
Description: 0001-Keep-the-socket-type-in-a-member-of-socket_base_t-an.patch

_______________________________________________
zeromq-dev mailing list
[email protected]
http://lists.zeromq.org/mailman/listinfo/zeromq-dev

Reply via email to