Author: hwright Date: Wed Oct 19 15:49:49 2011 New Revision: 1186283 URL: http://svn.apache.org/viewvc?rev=1186283&view=rev Log: No functional change.
* subversion/libsvn_fs_fs/fs_fs.c (set_revision_proplist): Remove a unneeded if-statement, and a superfluous return statement. Modified: subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c Modified: subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c?rev=1186283&r1=1186282&r2=1186283&view=diff ============================================================================== --- subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c (original) +++ subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c Wed Oct 19 15:49:49 2011 @@ -2905,32 +2905,29 @@ set_revision_proplist(svn_fs_t *fs, apr_hash_t *proplist, apr_pool_t *pool) { + const char *final_path; + const char *tmp_path; + const char *perms_reference; + svn_stream_t *stream; + SVN_ERR(ensure_revision_exists(fs, rev, pool)); - if (1) - { - const char *final_path = path_revprops(fs, rev, pool); - const char *tmp_path; - const char *perms_reference; - svn_stream_t *stream; - - /* ### do we have a directory sitting around already? we really shouldn't - ### have to get the dirname here. */ - SVN_ERR(svn_stream_open_unique(&stream, &tmp_path, - svn_dirent_dirname(final_path, pool), - svn_io_file_del_none, pool, pool)); - SVN_ERR(svn_hash_write2(proplist, stream, SVN_HASH_TERMINATOR, pool)); - SVN_ERR(svn_stream_close(stream)); + final_path = path_revprops(fs, rev, pool); - /* We use the rev file of this revision as the perms reference, - because when setting revprops for the first time, the revprop - file won't exist and therefore can't serve as its own reference. - (Whereas the rev file should already exist at this point.) */ - SVN_ERR(svn_fs_fs__path_rev_absolute(&perms_reference, fs, rev, pool)); - SVN_ERR(move_into_place(tmp_path, final_path, perms_reference, pool)); + /* ### do we have a directory sitting around already? we really shouldn't + ### have to get the dirname here. */ + SVN_ERR(svn_stream_open_unique(&stream, &tmp_path, + svn_dirent_dirname(final_path, pool), + svn_io_file_del_none, pool, pool)); + SVN_ERR(svn_hash_write2(proplist, stream, SVN_HASH_TERMINATOR, pool)); + SVN_ERR(svn_stream_close(stream)); - return SVN_NO_ERROR; - } + /* We use the rev file of this revision as the perms reference, + because when setting revprops for the first time, the revprop + file won't exist and therefore can't serve as its own reference. + (Whereas the rev file should already exist at this point.) */ + SVN_ERR(svn_fs_fs__path_rev_absolute(&perms_reference, fs, rev, pool)); + SVN_ERR(move_into_place(tmp_path, final_path, perms_reference, pool)); return SVN_NO_ERROR; }