You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by iv...@apache.org on 2015/05/13 17:37:18 UTC
svn commit: r1679228 - /subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
Author: ivan
Date: Wed May 13 15:37:17 2015
New Revision: 1679228
URL: http://svn.apache.org/r1679228
Log:
Follow-up to r1679269: Use canonical FS path in test.
* subversion/tests/libsvn_fs/fs-test.c
(freeze_and_commit): Use "/" instead of "" as PATH argument for
svn_fs_change_node_prop().
Modified:
subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
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=1679228&r1=1679227&r2=1679228&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/libsvn_fs/fs-test.c (original)
+++ subversion/trunk/subversion/tests/libsvn_fs/fs-test.c Wed May 13 15:37:17 2015
@@ -6930,7 +6930,7 @@ freeze_and_commit(const svn_test_opts_t
/* Make some commit. */
SVN_ERR(svn_fs_begin_txn(&txn, fs, new_rev, pool));
SVN_ERR(svn_fs_txn_root(&txn_root, txn, pool));
- SVN_ERR(svn_fs_change_node_prop(txn_root, "", "temperature",
+ SVN_ERR(svn_fs_change_node_prop(txn_root, "/", "temperature",
svn_string_create("310.05", pool),
pool));
SVN_ERR(test_commit_txn(&new_rev, txn, NULL, pool));
Re: svn commit: r1679228 - /subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
Posted by Ivan Zhakov <iv...@visualsvn.com>.
On 18 May 2015 at 02:25, Daniel Shahaf <d....@daniel.shahaf.name> wrote:
> Ivan Zhakov wrote on Fri, May 15, 2015 at 14:31:13 +0300:
>> On 14 May 2015 at 17:20, Daniel Shahaf <d....@daniel.shahaf.name> wrote:
>> > ivan@apache.org wrote on Wed, May 13, 2015 at 15:37:18 -0000:
>> >> * subversion/tests/libsvn_fs/fs-test.c
>> >> (freeze_and_commit): Use "/" instead of "" as PATH argument for
>> >> svn_fs_change_node_prop().
>> >
>> > What was the reason for the change? The FS API explicitly accepts both
>> > "" and "/" as valid spellings of the root directory. (See svn_fs.h:1438)
>> >
>> You're right, but I think using canonical FS path is better for
>> consistency: all other tests if this file uses canonical FS paths.
>
> Actually, some tests use "" and others use "/". I think that's exactly
> right: both syntaxes are valid inputs and both of them have test coverage.
Makes sense. Reverted my change in r1680346.
--
Ivan Zhakov
Re: svn commit: r1679228 -
/subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
Ivan Zhakov wrote on Fri, May 15, 2015 at 14:31:13 +0300:
> On 14 May 2015 at 17:20, Daniel Shahaf <d....@daniel.shahaf.name> wrote:
> > ivan@apache.org wrote on Wed, May 13, 2015 at 15:37:18 -0000:
> >> * subversion/tests/libsvn_fs/fs-test.c
> >> (freeze_and_commit): Use "/" instead of "" as PATH argument for
> >> svn_fs_change_node_prop().
> >
> > What was the reason for the change? The FS API explicitly accepts both
> > "" and "/" as valid spellings of the root directory. (See svn_fs.h:1438)
> >
> You're right, but I think using canonical FS path is better for
> consistency: all other tests if this file uses canonical FS paths.
Actually, some tests use "" and others use "/". I think that's exactly
right: both syntaxes are valid inputs and both of them have test coverage.
Re: svn commit: r1679228 - /subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
Posted by Ivan Zhakov <iv...@visualsvn.com>.
On 14 May 2015 at 17:20, Daniel Shahaf <d....@daniel.shahaf.name> wrote:
> ivan@apache.org wrote on Wed, May 13, 2015 at 15:37:18 -0000:
>> * subversion/tests/libsvn_fs/fs-test.c
>> (freeze_and_commit): Use "/" instead of "" as PATH argument for
>> svn_fs_change_node_prop().
>
> What was the reason for the change? The FS API explicitly accepts both
> "" and "/" as valid spellings of the root directory. (See svn_fs.h:1438)
>
You're right, but I think using canonical FS path is better for
consistency: all other tests if this file uses canonical FS paths.
--
Ivan Zhakov
Re: svn commit: r1679228 -
/subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
ivan@apache.org wrote on Wed, May 13, 2015 at 15:37:18 -0000:
> * subversion/tests/libsvn_fs/fs-test.c
> (freeze_and_commit): Use "/" instead of "" as PATH argument for
> svn_fs_change_node_prop().
What was the reason for the change? The FS API explicitly accepts both
"" and "/" as valid spellings of the root directory. (See svn_fs.h:1438)
Cheers,
Daniel
Re: svn commit: r1679228 -
/subversion/trunk/subversion/tests/libsvn_fs/fs-test.c
Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
ivan@apache.org wrote on Wed, May 13, 2015 at 15:37:18 -0000:
> * subversion/tests/libsvn_fs/fs-test.c
> (freeze_and_commit): Use "/" instead of "" as PATH argument for
> svn_fs_change_node_prop().
What was the reason for the change? The FS API explicitly accepts both
"" and "/" as valid spellings of the root directory. (See svn_fs.h:1438)
Cheers,
Daniel