array_free calling convention is odd.

[65278]

We already talked a bit about array_free. I think the caller should be responsible for setting pointer to an array to NULL after freeing it, not array_free itself. I think we should keep standard C semantics.

// caller
array_free(replica);
replica = NULL;

And array_free should not accept pointer to a pointer to struct Array as it does now, but just pointer to struct Array (or just Array because of typedef) as other array_* functions:

void
array_free(Array array)

Originally posted by @zilder in #57 (comment)