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 */