[tor-commits] [tor/master] Trivial readability changes in mmap unit tests

nickm at torproject.org nickm at torproject.org
Fri Mar 9 16:30:59 UTC 2012


commit 3c5a3b33e824ce50147be5171eaa75f93c8ae968
Author: Esteban Manchado Velázquez <emanchado at demiurgo.org>
Date:   Mon Feb 6 19:45:52 2012 +0100

    Trivial readability changes in mmap unit tests
---
 src/test/test_util.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/test/test_util.c b/src/test/test_util.c
index 8177c8c..ee41584 100644
--- a/src/test/test_util.c
+++ b/src/test/test_util.c
@@ -1065,8 +1065,6 @@ test_util_mmap(void)
   test_assert(! mapping);
 
   write_str_to_file(fname1, "Short file.", 1);
-  write_bytes_to_file(fname2, buf, buflen, 1);
-  write_bytes_to_file(fname3, buf, 16384, 1);
 
   mapping = tor_mmap_file(fname1);
   test_assert(mapping);
@@ -1093,9 +1091,10 @@ test_util_mmap(void)
 
   /* Make sure that we fail to map a no-longer-existent file. */
   mapping = tor_mmap_file(fname1);
-  test_assert(mapping == NULL);
+  test_assert(! mapping);
 
   /* Now try a big file that stretches across a few pages and isn't aligned */
+  write_bytes_to_file(fname2, buf, buflen, 1);
   mapping = tor_mmap_file(fname2);
   test_assert(mapping);
   test_eq(mapping->size, buflen);
@@ -1104,6 +1103,7 @@ test_util_mmap(void)
   mapping = NULL;
 
   /* Now try a big aligned file. */
+  write_bytes_to_file(fname3, buf, 16384, 1);
   mapping = tor_mmap_file(fname3);
   test_assert(mapping);
   test_eq(mapping->size, 16384);





More information about the tor-commits mailing list