Update zmq.hpp

[juseless]

Line 706 constructor was:
monitor_t() : socketPtr(NULL), monitor_socket{NULL} {}
but should be:
monitor_t() : socketPtr(NULL), monitor_socket(NULL) {}

Note change of bracket types for monitor_socket parameter.

[mwpowellhtx]

On July 16, 2017 8:04:31 PM EDT, juseless ***@***.***> wrote: Line 706 constructor was: monitor_t() : socketPtr(NULL), monitor_socket{NULL} {} but should be: monitor_t() : socketPtr(NULL), monitor_socket(NULL) {} Note change of bracket types for monitor_socket parameter.

"change of bracket types"? That is no small thing. There's no telling what you may have been initializing otherwise.

…

You can view, comment on, or merge this pull request online at: #135 -- Commit Summary -- * Update zmq.hpp -- File Changes -- M zmq.hpp (2) -- Patch Links -- https://github.com/zeromq/cppzmq/pull/135.patch https://github.com/zeromq/cppzmq/pull/135.diff

-- Sent from my Android device with K-9 Mail. Please excuse my brevity.

[bluca]

Thanks.
@a4z FYI

[bluca]

@juseless for your next PRs please use the following format for commit messages:

Problem: X is broken

Solution: do Y and Z to fix X

[a4z]

thanks, after 6 years of C++11, using single brace-enclosed initializer became so natural ...