archive-tar: unindent write_tar_entry by one level
It's used to be if (!sha1) { ... } else if (!path) { ... } else { ... } Now that the first two blocks are no-op. We can remove the if/else skeleton and put the else block back by one indent level. Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:

committed by
Junio C Hamano

parent
d240d41021
commit
853907097a
@ -170,11 +170,6 @@ static int write_tar_entry(struct archiver_args *args,
|
|||||||
|
|
||||||
memset(&header, 0, sizeof(header));
|
memset(&header, 0, sizeof(header));
|
||||||
|
|
||||||
if (!sha1) {
|
|
||||||
die("BUG: sha1 == NULL is not supported");
|
|
||||||
} else if (!path) {
|
|
||||||
die("BUG: path == NULL is not supported");
|
|
||||||
} else {
|
|
||||||
if (S_ISDIR(mode) || S_ISGITLINK(mode)) {
|
if (S_ISDIR(mode) || S_ISGITLINK(mode)) {
|
||||||
*header.typeflag = TYPEFLAG_DIR;
|
*header.typeflag = TYPEFLAG_DIR;
|
||||||
mode = (mode | 0777) & ~tar_umask;
|
mode = (mode | 0777) & ~tar_umask;
|
||||||
@ -203,7 +198,6 @@ static int write_tar_entry(struct archiver_args *args,
|
|||||||
}
|
}
|
||||||
} else
|
} else
|
||||||
memcpy(header.name, path, pathlen);
|
memcpy(header.name, path, pathlen);
|
||||||
}
|
|
||||||
|
|
||||||
if (S_ISLNK(mode) && buffer) {
|
if (S_ISLNK(mode) && buffer) {
|
||||||
if (size > sizeof(header.linkname)) {
|
if (size > sizeof(header.linkname)) {
|
||||||
|
Reference in New Issue
Block a user