You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by jo...@apache.org on 2004/05/24 08:06:20 UTC

cvs commit: apr/test testlfs.c

jorton      2004/05/23 23:06:20

  Modified:    test     testlfs.c
  Log:
  * test/testlfs.c (PRECOND): Return after ENOTIMPL checks trigger to
  prevent spurious test failures since ABTS_NOT_IMPL doesn't longjmp
  away.
  (test_open): Only set 'madefile' on success.
  
  Revision  Changes    Path
  1.7       +2 -2      apr/test/testlfs.c
  
  Index: testlfs.c
  ===================================================================
  RCS file: /home/cvs/apr/test/testlfs.c,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -d -u -r1.6 -r1.7
  --- testlfs.c	14 May 2004 14:43:22 -0000	1.6
  +++ testlfs.c	24 May 2004 06:06:20 -0000	1.7
  @@ -35,7 +35,7 @@
   
   static int madefile = 0;
   
  -#define PRECOND if (!madefile) ABTS_NOT_IMPL(tc, "Large file tests not enabled")
  +#define PRECOND if (!madefile) { ABTS_NOT_IMPL(tc, "Large file tests not enabled"); return; }
   
   #define TESTDIR "lfstests"
   #define TESTFILE "large.bin"
  @@ -73,7 +73,7 @@
           apr_assert_success(tc, "truncate file to 8gb", rv);
       }
   
  -    madefile = 1;
  +    madefile = rv == APR_SUCCESS;
   }
   
   static void test_reopen(abts_case *tc, void *data)