You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2021/04/23 17:50:12 UTC
svn commit: r1889133 - /subversion/trunk/subversion/tests/svn_test_main.c
Author: danielsh
Date: Fri Apr 23 17:50:11 2021
New Revision: 1889133
URL: http://svn.apache.org/viewvc?rev=1889133&view=rev
Log:
* subversion/tests/svn_test_main.c
(svn_test_main): Add a safety net ensuring APR_HAS_THREADS-less builds
would fail, rather than pass silently, if they tried to take the
"Run tests concurrently" codepath (which try would be a bug).
Review by: hartmannathan
Modified:
subversion/trunk/subversion/tests/svn_test_main.c
Modified: subversion/trunk/subversion/tests/svn_test_main.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/svn_test_main.c?rev=1889133&r1=1889132&r2=1889133&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/svn_test_main.c (original)
+++ subversion/trunk/subversion/tests/svn_test_main.c Fri Apr 23 17:50:11 2021
@@ -1089,9 +1089,9 @@ svn_test_main(int argc, const char *argv
svn_pool_clear(cleanup_pool);
}
}
-#if APR_HAS_THREADS
else
{
+#if APR_HAS_THREADS
got_error = do_tests_concurrently(opts.prog_name, test_funcs,
array_size, max_threads,
&opts, test_pool);
@@ -1099,8 +1099,11 @@ svn_test_main(int argc, const char *argv
/* Execute all cleanups */
svn_pool_clear(test_pool);
svn_pool_clear(cleanup_pool);
- }
+#else
+ /* Can't happen */
+ SVN_ERR_MALFUNCTION();
#endif
+ }
}
/* Clean up APR */