[zeromq-dev] [PATCH] Introduces bi-directional pipes

Martin Sustrik sustrik at 250bpm.com
Mon May 23 07:54:55 CEST 2011


Hi Fabien,

Thanks for review!

> 1. I don't think you should leave a printf in the code (in owns.cpp).

Oops!

> 2. Based on past experience with a similar structure, you should used a
> class policy instead of the N parameter in the array_t template.  It
> would allow more flexible and clearer code.  No need to remember what
> index_array1 and index_array2 mean.

It's cheap and nasty solution for the existing problem, namely that 
pipes have to be stored in two arrays at the same time, with all the 
array operations having O(1) complexity.

If you have a more elegant design, feel free to propose it.

Martin



More information about the zeromq-dev mailing list