[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen master] libfsimage: fix clang 10 build
commit e54c433adf01a242bf6e9fe9378a2c83d3f8b419 Author: Roger Pau Monne <roger.pau@xxxxxxxxxx> AuthorDate: Fri Mar 13 09:45:57 2020 +0100 Commit: Wei Liu <wl@xxxxxxx> CommitDate: Fri Mar 13 10:20:21 2020 +0000 libfsimage: fix clang 10 build clang complains with: fsys_zfs.c:826:2: error: converting the enum constant to a boolean [-Werror,-Wint-in-bool-context] VERIFY_DN_TYPE(dn, DMU_OT_PLAIN_FILE_CONTENTS); ^ /wrkdirs/usr/ports/sysutils/xen-tools/work/xen-4.13.0/tools/libfsimage/zfs/../../../tools/libfsimage/zfs/fsys_zfs.h:74:11: note: expanded from macro 'VERIFY_DN_TYPE' if (type && (dnp)->dn_type != type) { \ ^ 1 error generated. Fix this by not forcing an implicit conversion of the enum into a boolean and instead comparing with the 0 enumerator. Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx> Acked-by: Wei Liu <wl@xxxxxxx> --- tools/libfsimage/zfs/fsys_zfs.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/libfsimage/zfs/fsys_zfs.h b/tools/libfsimage/zfs/fsys_zfs.h index 5cd627dbac..721972a05a 100644 --- a/tools/libfsimage/zfs/fsys_zfs.h +++ b/tools/libfsimage/zfs/fsys_zfs.h @@ -71,7 +71,7 @@ typedef unsigned int size_t; * Can only be used in functions returning non-0 for failure. */ #define VERIFY_DN_TYPE(dnp, type) \ - if (type && (dnp)->dn_type != type) { \ + if (type != DMU_OT_NONE && (dnp)->dn_type != type) { \ return (ERR_FSYS_CORRUPT); \ } @@ -80,7 +80,7 @@ typedef unsigned int size_t; * Can only be used in functions returning 0 for failure. */ #define VERIFY_OS_TYPE(osp, type) \ - if (type && (osp)->os_type != type) { \ + if (type != DMU_OST_NONE && (osp)->os_type != type) { \ errnum = ERR_FSYS_CORRUPT; \ return (0); \ } -- generated by git-patchbot for /home/xen/git/xen.git#master _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |