Actually, don't worry about this. I did it this morning, and I need to do a
release this weekend, to support MikePurvis. It will be checked into the
repo shortly.

Paul


On Sat, Sep 21, 2013 at 8:36 AM, Lorenzo Riano <notifications@github.com>wrote:

> In rosserial_client/src/ros_lib/ros/node_handle.h:
>
> > @@ -115,7 +115,19 @@
> >         * Setup Functions
> >         */
> >      public:
> > -      NodeHandle_() : configured_(false) {}
> > +      NodeHandle_() : configured_(false) {
> > +        memset(publishers, 0, sizeof(publishers));
>
> Paul, I'll do it next week as I'm off for the weekend
> … <#14140f38fb8260e0_>
> On Sep 20, 2013 5:43 AM, "PaulBouchier" <notifications@github.com> wrote:
> In rosserial_client/src/ros_lib/ros/node_handle.h: > @@ -115,7 +115,19 @@ >
> * Setup Functions > */ > public: > - NodeHandle_() : configured_(false) {}
> > + NodeHandle_() : configured_(false) { > + memset(publishers, 0,
> sizeof(publishers)); Lorenzo, if you could replace the calls to memcpy with
> a for-loop that will address Mike's concerns about linking with memcpy. for
> (int i=0, ptr=beginAddress; i<sizeof(struct); i++,ptr++) *ptr=0; I'll merge
> & do a release this weekend. — Reply to this email directly or view it on
> GitHub<https://github.com/ros-drivers/rosserial/pull/62/files#r6485424> .
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ros-drivers/rosserial/pull/62/files#r6502732>
> .
>