From e3ef871d6a80d72db02c9ab1ef492e8f58c2ddeb Mon Sep 17 00:00:00 2001 From: David Oberhollenzer Date: Tue, 16 Jul 2019 21:02:58 +0200 Subject: cleanup: move error handling into read_retry If read_retry fails to read the expected amount of data (EOF or otherwise), it is almost always an error. This commit renames read_retry to read_data and moves error handling into the function, making a lot of error handling code redundant. Signed-off-by: David Oberhollenzer --- tests/tar_pax.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'tests/tar_pax.c') diff --git a/tests/tar_pax.c b/tests/tar_pax.c index 6ecc361..800e9f1 100644 --- a/tests/tar_pax.c +++ b/tests/tar_pax.c @@ -50,7 +50,7 @@ int main(void) assert(hdr.sb.st_ctime == 1542905892); assert(strcmp(hdr.name, "input.txt") == 0); assert(!hdr.unknown_record); - assert(read_retry(fd, buffer, 5) == 5); + assert(read_data("data0", fd, buffer, 5) == 0); buffer[5] = '\0'; assert(strcmp(buffer, "test\n") == 0); clear_header(&hdr); @@ -81,7 +81,7 @@ int main(void) assert(hdr.sb.st_ctime == 1542999260); assert(strcmp(hdr.name, "input.txt") == 0); assert(!hdr.unknown_record); - assert(read_retry(fd, buffer, 5) == 5); + assert(read_data("data1", fd, buffer, 5) == 0); buffer[5] = '\0'; assert(strcmp(buffer, "test\n") == 0); clear_header(&hdr); @@ -98,7 +98,7 @@ int main(void) assert(hdr.sb.st_ctime == 1543015033); assert(strcmp(hdr.name, "input.txt") == 0); assert(!hdr.unknown_record); - assert(read_retry(fd, buffer, 5) == 5); + assert(read_data("data2", fd, buffer, 5) == 0); buffer[5] = '\0'; assert(strcmp(buffer, "test\n") == 0); clear_header(&hdr); @@ -115,7 +115,7 @@ int main(void) assert(hdr.sb.st_ctime == 1543015908); assert(strcmp(hdr.name, "input.txt") == 0); assert(!hdr.unknown_record); - assert(read_retry(fd, buffer, 5) == 5); + assert(read_data("data3", fd, buffer, 5) == 0); buffer[5] = '\0'; assert(strcmp(buffer, "test\n") == 0); clear_header(&hdr); @@ -132,7 +132,7 @@ int main(void) assert(hdr.sb.st_ctime == 1542909670); assert(strcmp(hdr.name, filename) == 0); assert(!hdr.unknown_record); - assert(read_retry(fd, buffer, 5) == 5); + assert(read_data("data4", fd, buffer, 5) == 0); buffer[5] = '\0'; assert(strcmp(buffer, "test\n") == 0); clear_header(&hdr); -- cgit v1.2.3