Addressed review feedback
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61
diff --git a/src/indexer.c b/src/indexer.c
index 9315362..3d1c81c 100644
--- a/src/indexer.c
+++ b/src/indexer.c
@@ -151,11 +151,11 @@ cleanup:
if (fd != -1)
p_close(fd);
- if (git_buf_is_allocated(&tmp_path))
- (void)p_unlink(git_buf_cstr(&tmp_path));
+ if (git_buf_len(&tmp_path) > 0)
+ p_unlink(git_buf_cstr(&tmp_path));
if (idx->pack != NULL)
- (void)p_unlink(idx->pack->pack_name);
+ p_unlink(idx->pack->pack_name);
git_buf_free(&path);
git_buf_free(&tmp_path);
@@ -1094,10 +1094,10 @@ void git_indexer_free(git_indexer *idx)
git_mwindow_free_all(&idx->pack->mwf);
/* We need to close the descriptor here so Windows doesn't choke on unlink */
if (idx->pack->mwf.fd != -1) {
- (void)p_close(idx->pack->mwf.fd);
+ p_close(idx->pack->mwf.fd);
idx->pack->mwf.fd = -1;
}
- (void)p_unlink(idx->pack->pack_name);
+ p_unlink(idx->pack->pack_name);
if (!git_mutex_lock(&git__mwindow_mutex)) {
git_packfile_free(idx->pack);
diff --git a/tests/pack/indexer.c b/tests/pack/indexer.c
index 2c97246..1e514b2 100644
--- a/tests/pack/indexer.c
+++ b/tests/pack/indexer.c
@@ -155,11 +155,7 @@ void test_pack_indexer__no_tmp_files(void)
cl_git_pass(git_buf_sets(&path, clar_sandbox_path()));
cl_git_pass(find_tmp_file_recurs(&first_tmp_file, &path));
git_buf_free(&path);
- if (git_buf_is_allocated(&first_tmp_file)) {
- git_buf_free(&first_tmp_file);
- cl_warning("Found a temporary file before running the test");
- cl_skip();
- }
+ cl_assert(git_buf_len(&first_tmp_file) == 0);
cl_git_pass(git_indexer_new(&idx, ".", 0, NULL, NULL, NULL));
git_indexer_free(idx);
@@ -167,9 +163,6 @@ void test_pack_indexer__no_tmp_files(void)
cl_git_pass(git_buf_sets(&path, clar_sandbox_path()));
cl_git_pass(find_tmp_file_recurs(&first_tmp_file, &path));
git_buf_free(&path);
- if (git_buf_is_allocated(&first_tmp_file)) {
- cl_warning(git_buf_cstr(&first_tmp_file));
- git_buf_free(&first_tmp_file);
- cl_fail("Found a temporary file");
- }
+ cl_assert(git_buf_len(&first_tmp_file) == 0);
+ git_buf_free(&first_tmp_file);
}