]> git.ipfire.org Git - thirdparty/libarchive.git/commit
Fix the following -Werror warning from clang 3.5.0, while building cpio.c on amd64... 94/head
authorDimitry Andric <dimitry@andric.com>
Sat, 22 Nov 2014 12:01:08 +0000 (13:01 +0100)
committerDimitry Andric <dimitry@andric.com>
Sat, 22 Nov 2014 12:01:08 +0000 (13:01 +0100)
commitf9d04ce68d136e1a5276ea1a764f2ec99772423d
tree4065c28f6055475b0c9cd0c7ecb9a3ccefa090c0
parentacaeb910dcbfc062b6e0549f7cfe62073278ad55
Fix the following -Werror warning from clang 3.5.0, while building cpio.c on amd64 (or any arch with 64-bit time_t):

libarchive/cpio/cpio.c:1143:6: error: absolute value function 'abs' given an argument of type 'long' but has parameter of type 'int' which may cause truncation of value [-Werror,-Wabsolute-value]
        if (abs(mtime - now) > (365/2)*86400)
            ^
libarchive/cpio/cpio.c:1143:6: note: use function 'labs' instead
        if (abs(mtime - now) > (365/2)*86400)
            ^~~
            labs
1 error generated.

This is because time_t is a long on amd64. To avoid the warning, just copy the equivalent test from a few lines before, which is used in the Windows case, and which is type safe.

Obtained from: https://reviews.freebsd.org/D1198
cpio/cpio.c