Better comments

Author: asomers

test/test_unistd.rs

@@ -15,6 +15,16 @@ use tempfile::tempfile;
 use tempdir::TempDir;
 use libc::off_t;
 
+/// Run a test in a subprocess
+///
+/// This is useful for tests that change a process-level property, like the cwd
+/// or a signal handler.
+///
+/// # Arguments
+///
+/// * `func` - function to run in the subprocess.  It should exit 0 on success
+/// or nonzero on failure.  The usual `assert` functions will do the right
+/// thing.
 fn test_in_subprocess<F: Fn()>(func: F) {
     let pid = fork();
     match pid {
@@ -190,7 +200,6 @@ fn test_getcwd() {
         let mut inner_tmp_dir = tmp_dir.path().to_path_buf();
         for _ in 0..5 {
             let newdir = iter::repeat("a").take(100).collect::<String>();
-            //inner_tmp_dir = inner_tmp_dir.join(newdir).path();
             inner_tmp_dir.push(newdir);
             assert!(mkdir(inner_tmp_dir.as_path(), stat::S_IRWXU).is_ok());
         }