Author: ivan Date: Wed Aug 17 16:37:14 2016 New Revision: 1756651 URL: http://svn.apache.org/viewvc?rev=1756651&view=rev Log: Follow-up to r1755486: Rename svn_stream_checksum() to svn_stream_contents_checksum().
Suggested by: danielsh * subversion/include/svn_io.h * subversion/libsvn_subr/stream.c (svn_stream_checksum): Rename to svn_stream_contents_checksum(). (compute_stream_checksum): Update docstring. * subversion/libsvn_fs/fs-loader.c * subversion/libsvn_subr/io.c * subversion/tests/libsvn_fs/fs-test.c * subversion/tests/libsvn_subr/stream-test.c (svn_fs_file_checksum, svn_io_file_checksum2, get_file_checksum, test_stream_checksum): Adapt callers. Modified: subversion/trunk/subversion/include/svn_io.h subversion/trunk/subversion/libsvn_fs/fs-loader.c subversion/trunk/subversion/libsvn_subr/io.c subversion/trunk/subversion/libsvn_subr/stream.c subversion/trunk/subversion/tests/libsvn_fs/fs-test.c subversion/trunk/subversion/tests/libsvn_subr/stream-test.c Modified: subversion/trunk/subversion/include/svn_io.h URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_io.h?rev=1756651&r1=1756650&r2=1756651&view=diff ============================================================================== --- subversion/trunk/subversion/include/svn_io.h (original) +++ subversion/trunk/subversion/include/svn_io.h Wed Aug 17 16:37:14 2016 @@ -1266,11 +1266,11 @@ svn_stream_checksummed(svn_stream_t *str * @since New in 1.10. */ svn_error_t * -svn_stream_checksum(svn_checksum_t **checksum, - svn_stream_t *stream, - svn_checksum_kind_t kind, - apr_pool_t *result_pool, - apr_pool_t *scratch_pool); +svn_stream_contents_checksum(svn_checksum_t **checksum, + svn_stream_t *stream, + svn_checksum_kind_t kind, + apr_pool_t *result_pool, + apr_pool_t *scratch_pool); /** Read from a generic stream until @a buffer is filled upto @a *len or * until EOF is reached. @see svn_stream_t Modified: subversion/trunk/subversion/libsvn_fs/fs-loader.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs/fs-loader.c?rev=1756651&r1=1756650&r2=1756651&view=diff ============================================================================== --- subversion/trunk/subversion/libsvn_fs/fs-loader.c (original) +++ subversion/trunk/subversion/libsvn_fs/fs-loader.c Wed Aug 17 16:37:14 2016 @@ -1504,7 +1504,8 @@ svn_fs_file_checksum(svn_checksum_t **ch svn_stream_t *contents; SVN_ERR(svn_fs_file_contents(&contents, root, path, pool)); - SVN_ERR(svn_stream_checksum(checksum, contents, kind, pool, pool)); + SVN_ERR(svn_stream_contents_checksum(checksum, contents, kind, + pool, pool)); } return SVN_NO_ERROR; Modified: subversion/trunk/subversion/libsvn_subr/io.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_subr/io.c?rev=1756651&r1=1756650&r2=1756651&view=diff ============================================================================== --- subversion/trunk/subversion/libsvn_subr/io.c (original) +++ subversion/trunk/subversion/libsvn_subr/io.c Wed Aug 17 16:37:14 2016 @@ -1482,7 +1482,8 @@ svn_io_file_checksum2(svn_checksum_t **c SVN_ERR(svn_io_file_open(&f, file, APR_READ, APR_OS_DEFAULT, pool)); file_stream = svn_stream_from_aprfile2(f, FALSE, pool); - SVN_ERR(svn_stream_checksum(checksum, file_stream, kind, pool, pool)); + SVN_ERR(svn_stream_contents_checksum(checksum, file_stream, kind, + pool, pool)); return SVN_NO_ERROR; } Modified: subversion/trunk/subversion/libsvn_subr/stream.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_subr/stream.c?rev=1756651&r1=1756650&r2=1756651&view=diff ============================================================================== --- subversion/trunk/subversion/libsvn_subr/stream.c (original) +++ subversion/trunk/subversion/libsvn_subr/stream.c Wed Aug 17 16:37:14 2016 @@ -1478,8 +1478,8 @@ svn_stream_checksummed2(svn_stream_t *st return s; } -/* Helper for svn_stream_checksum() to compute checksum of KIND of STREAM. - * This function doesn't close source stream. */ +/* Helper for svn_stream_contents_checksum() to compute checksum of + * KIND of STREAM. This function doesn't close source stream. */ static svn_error_t * compute_stream_checksum(svn_checksum_t **checksum, svn_stream_t *stream, @@ -1508,11 +1508,11 @@ compute_stream_checksum(svn_checksum_t * } svn_error_t * -svn_stream_checksum(svn_checksum_t **checksum, - svn_stream_t *stream, - svn_checksum_kind_t kind, - apr_pool_t *result_pool, - apr_pool_t *scratch_pool) +svn_stream_contents_checksum(svn_checksum_t **checksum, + svn_stream_t *stream, + svn_checksum_kind_t kind, + apr_pool_t *result_pool, + apr_pool_t *scratch_pool) { svn_error_t *err; Modified: subversion/trunk/subversion/tests/libsvn_fs/fs-test.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/libsvn_fs/fs-test.c?rev=1756651&r1=1756650&r2=1756651&view=diff ============================================================================== --- subversion/trunk/subversion/tests/libsvn_fs/fs-test.c (original) +++ subversion/trunk/subversion/tests/libsvn_fs/fs-test.c Wed Aug 17 16:37:14 2016 @@ -3596,7 +3596,8 @@ get_file_checksum(svn_checksum_t **check /* Get a stream for the file contents. */ SVN_ERR(svn_fs_file_contents(&stream, root, path, pool)); - SVN_ERR(svn_stream_checksum(checksum, stream, checksum_kind, pool, pool)); + SVN_ERR(svn_stream_contents_checksum(checksum, stream, checksum_kind, + pool, pool)); return SVN_NO_ERROR; } Modified: subversion/trunk/subversion/tests/libsvn_subr/stream-test.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/libsvn_subr/stream-test.c?rev=1756651&r1=1756650&r2=1756651&view=diff ============================================================================== --- subversion/trunk/subversion/tests/libsvn_subr/stream-test.c (original) +++ subversion/trunk/subversion/tests/libsvn_subr/stream-test.c Wed Aug 17 16:37:14 2016 @@ -855,13 +855,15 @@ test_stream_checksum(apr_pool_t *pool) svn_string_create("The quick brown fox jumps over the lazy dog", pool); svn_checksum_t *actual; - SVN_ERR(svn_stream_checksum(&actual, svn_stream_from_string(str, pool), - svn_checksum_md5, pool, pool)); + SVN_ERR(svn_stream_contents_checksum(&actual, + svn_stream_from_string(str, pool), + svn_checksum_md5, pool, pool)); SVN_TEST_STRING_ASSERT("9e107d9d372bb6826bd81d3542a419d6", svn_checksum_to_cstring(actual, pool)); - SVN_ERR(svn_stream_checksum(&actual, svn_stream_from_string(str, pool), - svn_checksum_sha1, pool, pool)); + SVN_ERR(svn_stream_contents_checksum(&actual, + svn_stream_from_string(str, pool), + svn_checksum_sha1, pool, pool)); SVN_TEST_STRING_ASSERT("2fd4e1c67a2d28fced849ee1bb76e7391b93eb12", svn_checksum_to_cstring(actual, pool)); @@ -900,7 +902,7 @@ static struct svn_test_descriptor_t test SVN_TEST_PASS2(test_stream_compressed_read_full, "test compression for streams without partial read"), SVN_TEST_PASS2(test_stream_checksum, - "test svn_stream_checksum()"), + "test svn_stream_contents_checksum()"), SVN_TEST_NULL };