(create_node): Don't take the node's content as an argument, deduce it from the other arguments instead.  Take a trail instead of a pool, since we need a trail for get_node_revision, and all callers have one handy anyway.  Sanity check arguments using assert() instead of the full run-time error system.  All callers changed.
(create_node): Don't take the node's content as an argument, deduce it
from the other arguments instead.  Take a trail instead of a pool,
since we need a trail for get_node_revision, and all callers have one
handy anyway.  Sanity check arguments using assert() instead of the
full run-time error system.  All callers changed.

(find_dir_entry): Promise that the returned entry is well-formed, and
that it is allocated in either PARENT->pool or TRAIL->pool.

(make_entry): Do sanity checks before any real work.

(svn_fs__dag_open): More helpful error message.

(svn_fs__dag_revision_root, svn_fs__dag_txn_root,
svn_fs__dag_clone_child, svn_fs__dag_clone_root, svn_fs__dag_open):
Simplify.

(node_is_kind_p): Inspect header correctly.  (Ben found this bug.)

(cache_node_revision, uncache_node_revision, get_node_revision,
set_node_revision, has_mutable_flag): Move earlier in the file.

jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion