← Commit list

9c60b2907

Glenn L McGrath
Glenn L McGrath
2002-11-03
FAILED Duration: 6s

Subject

Use xread_char to save a few bytes, fix indenting of comments

Binary

No binary (build failed)

Architecture & Compiler Flags

Architecture: x86_64-linux-gnu Compiler: gcc (Ubuntu 11.4.0-1ubuntu1~22.04.3) 11.4.0

—

Build Log

(log yok)

Commit Diff GitHub →

archival/libunarchive/check_header_gzip.c+12 -23
@@ -17,7 +17,7 @@ extern void check_header_gzip(int src_fd)
xread_all(src_fd, header.raw, 8);
- /* Check the compression method */
+ /* Check the compression method */
if (header.formated.method != 8) {
error_msg_and_die("Unknown compression method %d",
header.formated.method);
@@ -27,41 +27,30 @@ extern void check_header_gzip(int src_fd)
/* bit 2 set: extra field present */
unsigned char extra_short;
- extra_short = xread_char(src_fd);
- extra_short += xread_char(src_fd) << 8;
+ extra_short = xread_char(src_fd) + (xread_char(src_fd) << 8);
while (extra_short > 0) {
- /* Ignore extra field */
+ /* Ignore extra field */
xread_char(src_fd);
extra_short--;
}
}
- /* Discard original name if any */
+ /* Discard original name if any */
if (header.formated.flags & 0x08) {
- /* bit 3 set: original file name present */
- char tmp;
-
- do {
- read(src_fd, &tmp, 1);
- } while (tmp != 0);
+ /* bit 3 set: original file name present */
+ while(xread_char(src_fd) != 0);
}
- /* Discard file comment if any */
+ /* Discard file comment if any */
if (header.formated.flags & 0x10) {
- /* bit 4 set: file comment present */
- char tmp;
-
- do {
- read(src_fd, &tmp, 1);
- } while (tmp != 0);
+ /* bit 4 set: file comment present */
+ while(xread_char(src_fd) != 0);
}
- /* Read the header checksum */
+ /* Read the header checksum */
if (header.formated.flags & 0x02) {
- char tmp;
-
- read(src_fd, &tmp, 1);
- read(src_fd, &tmp, 1);
+ xread_char(src_fd);
+ xread_char(src_fd);
}
return;