Removed check for same mem allocator in S_can_contain.

This is too strict, as it prevents the use of dynamically
loaded extensions: see
commonmark#123 (comment).

Documented in man page and public header that one should use the same
memory allocator for every node in a tree.

Author: jgm

src/cmark.h

@@ -112,7 +112,8 @@ typedef struct cmark_mem {
 CMARK_EXPORT cmark_node *cmark_node_new(cmark_node_type type);
 
 /** Same as `cmark_node_new`, but explicitly listing the memory
- * allocator used to allocate the node
+ * allocator used to allocate the node.  Note:  be sure to use the same
+ * allocator for every node in a tree, or bad things can happen.
  */
 CMARK_EXPORT cmark_node *cmark_node_new_with_mem(cmark_node_type type, cmark_mem *mem);
 

src/node.c

@@ -30,9 +30,6 @@ static bool S_can_contain(cmark_node *node, cmark_node *child) {
   if (node == NULL || child == NULL) {
     return false;
   }
-  if (NODE_MEM(node) != NODE_MEM(child)) {
-    return 0;
-  }
 
   // Verify that child is not an ancestor of node or equal to node.
   cur = node;