Help the debugger, profiler and compiler a bit by replacing several uses of
svn_wc__db_with_txn() with the SVN_WC__DB_WITH_TXN() macro.

As this makes the function call the transaction function directory instead of
via the api this reduces the amount of (call-)stack used. The compiler can use
this to perform some inlining and calling method optimizations, while my
profiler hides more noise from its reports.

* subversion/libsvn_wc/wc_db.c
  (insert_base_node,
   insert_working_node): Use typed baton argument.

  (svn_wc__db_base_add_directory,
   svn_wc__db_base_add_incomplete_directory,
   svn_wc__db_base_add_incomplete_directory,
   svn_wc__db_base_add_symlink,
   add_excluded_or_not_present_node): Update calling.

  (insert_external_node): Use typed baton argument.
  (svn_wc__db_external_add_file,
   svn_wc__db_external_add_dir): Update calling.

  (svn_wc__db_op_copy_dir): Update calling.


git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1453805 13f79535-47bb-0310-9956-ffa450edef68