Yes - you are correct.  Good find!

Thanks,

Brian

On Mar 18, 2014, at 5:28 AM, Martin Hurton <notifications@github.com> wrote:

> In src/norm_engine.cpp:
> 
> > +            // Should we write message size header for NORM to use? Or expect NORM
> > +            // receiver to decode ZMQ message framing format(s)?
> > +            // OK - we need to use a byte to denote when the ZMQ frame is the _first_
> > +            //      frame of a message so it can be decoded properly when a receiver
> > +            //      'syncs' mid-stream.  We key off the the state of the 'more_flag'
> > +            //      I.e.,If  more_flag _was_ false previously, this is the first
> > +            //      frame of a ZMQ message.
> > +            if (tx_more_bit) 
> > +                tx_buffer[0] = (char)0xff;  // this is not first frame of message
> > +            else
> > +                tx_buffer[0] = 0x00;  // this is first frame of message
> > +            tx_more_bit = (0 != (tx_msg.flags() & v2_protocol_t::more_flag));
> > +            tx_len = 1;
> > +        }
> > +        // Get more data from encoder
> > +        size_t space = BUFFER_SIZE - tx_index;
> Should this be BUFFER_SIZE - tx_len rahter than BUFFER_SIZE - tx_index?
> 
> —
> Reply to this email directly or view it on GitHub.
>