Looks OK.  One minor formatting comment that I will fix myself when I
commit the patch.


Diego.


http://codereview.appspot.com/4532102/diff/1/pph-streamer-in.c
File pph-streamer-in.c (right):

http://codereview.appspot.com/4532102/diff/1/pph-streamer-in.c#newcode42
pph-streamer-in.c:42: pph_register_shared_data (STREAM, DATA, IX);      \
       (DATA) = (ALLOC_EXPR);                                   \
-      pph_stream_register_shared_data (STREAM, DATA, IX);      \
+      pph_register_shared_data (STREAM, DATA, IX);     \

Align the '\' with the ones above.

http://codereview.appspot.com/4532102/

Reply via email to