[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1499747 - /subversion/trunk/subversion/tests/libsvn_wc/op-depth-test.c

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Tue, 09 Jul 2013 10:01:04 +0100

"Bert Huijben" <bert_at_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
Received on 2013-07-09 11:01:50 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.