subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Philip Martin <philip.mar...@wandisco.com>
Subject Re: svn commit: r1499747 - /subversion/trunk/subversion/tests/libsvn_wc/op-depth-test.c
Date Tue, 09 Jul 2013 09:01:04 GMT
"Bert Huijben" <bert@qqmail.nl> writes:

>> @@ -1903,7 +1903,7 @@ check_db_actual(svn_test__sandbox_t* b,
>>      {
>>        const char *local_relpath
>>          = svn__apr_hash_index_key(apr_hash_first(b->pool, path_hash));
>> -      return svn_error_createf(SVN_ERR_TEST_FAILED,
>> svn_sqlite__close(sdb),
>> +      return svn_error_createf(SVN_ERR_TEST_FAILED,
>> svn_sqlite__reset(stmt),
>>                                 "actual '%s' expected", local_relpath);
>
> Are these two changes related to the test? Or is this just fixing
> existing bugs?

An existing bug that occurs when a test FAILs and runs the error code
path.  I probably should have made 2 commits.

-- 
Philip Martin | Subversion Committer
WANdisco | Non-Stop Data
www.wandisco.com

Mime
View raw message