Skip to content

Commit

Permalink
Merge branch 'jk/big-and-future-archive-tar'
Browse files Browse the repository at this point in the history
A small code clean-up.

* jk/big-and-future-archive-tar:
  archive-tar: make write_extended_header() void
  • Loading branch information
gitster committed Aug 12, 2016
2 parents 6d4960a + 560b0e8 commit 57734b4
Showing 1 changed file with 5 additions and 10 deletions.
15 changes: 5 additions & 10 deletions archive-tar.c
Original file line number Diff line number Diff line change
Expand Up @@ -213,9 +213,9 @@ static void prepare_header(struct archiver_args *args,
xsnprintf(header->chksum, sizeof(header->chksum), "%07o", ustar_header_chksum(header));
}

static int write_extended_header(struct archiver_args *args,
const unsigned char *sha1,
const void *buffer, unsigned long size)
static void write_extended_header(struct archiver_args *args,
const unsigned char *sha1,
const void *buffer, unsigned long size)
{
struct ustar_header header;
unsigned int mode;
Expand All @@ -226,7 +226,6 @@ static int write_extended_header(struct archiver_args *args,
prepare_header(args, &header, mode, size);
write_blocked(&header, sizeof(header));
write_blocked(buffer, size);
return 0;
}

static int write_tar_entry(struct archiver_args *args,
Expand Down Expand Up @@ -305,12 +304,8 @@ static int write_tar_entry(struct archiver_args *args,
prepare_header(args, &header, mode, size_in_header);

if (ext_header.len > 0) {
err = write_extended_header(args, sha1, ext_header.buf,
ext_header.len);
if (err) {
free(buffer);
return err;
}
write_extended_header(args, sha1, ext_header.buf,
ext_header.len);
}
strbuf_release(&ext_header);
write_blocked(&header, sizeof(header));
Expand Down

0 comments on commit 57734b4

Please sign in to comment.