On Tue, Dec 10, 2013 at 02:49:29PM +0100, Helmut Schaa wrote:
> Use a global array that gets initialized by constructor functions
> per coverage definition.
> 
> Signed-off-by: Helmut Schaa <helmut.sc...@googlemail.com>
> ---
> 
> The realloc is not the nicest thing to do but looks reasonable as initial
> approach. Using a list would cause more intrusive changes to the coverage
> implementation.

Could you refactor the constructor code as a call to a helper
function, please?  (That will allow you to make the coverage_counters
definitions static within coverage.c, I think.)

I suggest using x2nrealloc instead of xrealloc for this.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to