[zeromq-dev] [PATCH] Introduces bi-directional pipes
fabien.ninoles at gmail.com
Sun May 22 21:30:09 CEST 2011
Nice work, which certainly make some features easier.
Two remarks however:
1. I don't think you should leave a printf in the code (in owns.cpp).
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.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the zeromq-dev