Doug Cutting 2011-12-13, 18:02
-Re: review recent C patches?
Douglas Creager 2011-12-13, 19:04
> A number of C patches have been submitted to Avro recently.
> Can someone more familiar with the C implementation than I am please
> review these? If no one else does first, I'll try to later this week.
> My approach would be to check that:
> 0. They sound and look reasonable.
> 1. They include a test.
> 2. Tests pass after they are applied.
> If these are true then I'll commit them.
> Others would probably be much better at (0) than I am.
> As a project we should review patches promptly.
My apologies for not getting to them sooner. I'm taking a look at them now.
Doug Cutting 2011-12-13, 19:16