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/