OK - understood.  I have made the change in my local fork.


On Mar 18, 2014, at 5:49 AM, Martin Hurton <notifications@github.com> wrote:

> In src/norm_engine.cpp:
> 
> > +                zmq_output_ready = false;
> > +                break;
> > +            }
> > +            zmq_encoder.load_msg(&tx_msg);
> > +            // 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));
> Testing for more flag, please use msg_t::more.
> v2_protocol_t::more_flag is a representaion used on wire.
> 
> —
> Reply to this email directly or view it on GitHub.
>