Step 3 in removing repos-level checks using mergeinfo created on a copy: Remove the mergeinfo filter function argument and baton from svn_fs_get_mergeinfo_for_tree(). The filter was only used in one place, was a giant hack to begin with, and is now no longer needed.
* subversion/libsvn_repos/log.c (get_combined_mergeinfo): Remove filter and baton parameters for svn_fs_get_mergeinfo_for_tree().
* subversion/libsvn_fs_util/mergeinfo-sqlite-index.c (svn_fs_mergeinfo__get_mergeinfo_for_tree, get_mergeinfo_for_children): Stop using the filter parameters.
Wrap sqlite3_stmt inside a svn_fs__sqlite_stmt_t struct, which also includes a pointer to the db.
This has two positive results: - we stop using sqlite3_db_handle, which is relatively new to sqlite - we introduce a few new helpers for getting values from queries, which reduces the number of casts we make
* subversion/libsvn_fs_util/mergeinfo-sqlite-index.c (index_path_mergeinfo, table_has_any_rows_with_rev, parse_mergeinfo_from_db, get_mergeinfo_for_path, get_mergeinfo_for_children): Adjust. (Pool usage may need to be reaudited.)
* subversion/libsvn_fs_util/sqlite-util.h * subversion/libsvn_fs_util/sqlite-util.c (struct svn_fs__sqlite_stmt_t): New. (svn_fs__sqlite_prepare): Take a pool argument; return a struct containing the statement and the db. (step_with_expectation, svn_fs__sqlite_step_done, svn_fs__sqlite_step_row, svn_fs__sqlite_step, svn_fs__sqlite_bind_int64, svn_fs__sqlite_bind_text, svn_fs__sqlite_finalize, svn_fs__sqlite_reset, upgrade_format): Adjust. (svn_fs__sqlite_reset): Remove a random declaration. (svn_fs__sqlite_column_text, svn_fs__sqlite_column_revnum, svn_fs__sqlite_column_boolean, svn_fs__sqlite_column_int): New.