From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1791) id C95A4389364C; Thu, 28 Sep 2023 17:51:17 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C95A4389364C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1695923477; bh=yDiJ2oo7UR1sJpC3ABc5MXJqkbGOhr4BDcuPmZA4Gug=; h=From:To:Subject:Date:From; b=rpBXdXUR6wr/mvSK1LTNZBOeuf1xd4sRtzpt+N9zDLavnzLk8YyJeHi5GMAGNSg1L qC6UzTu1E4OJXuUmV7Azt0eT17tAi+y1ormIwLEcmnIC8r2Q7xzBnw9F/BiCmwnhJG RbRlMDHajCY/jz5Kj82yDzE1GFDmXu1jvs2US3xY= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Adhemerval Zanella To: glibc-cvs@sourceware.org Subject: [glibc/azanella/clang] timezone: Fix clang operator precedence warning X-Act-Checkin: glibc X-Git-Author: Adhemerval Zanella X-Git-Refname: refs/heads/azanella/clang X-Git-Oldrev: e4000a8fe7c372d480e2ab7aec4e909815856621 X-Git-Newrev: e2a1ef0422b996e55e205b2c19b50dea6db64a36 Message-Id: <20230928175117.C95A4389364C@sourceware.org> Date: Thu, 28 Sep 2023 17:51:17 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=e2a1ef0422b996e55e205b2c19b50dea6db64a36 commit e2a1ef0422b996e55e205b2c19b50dea6db64a36 Author: Adhemerval Zanella Date: Thu Mar 10 16:44:45 2022 -0300 timezone: Fix clang operator precedence warning clang issues: zdump.c:669:7: error: operator '?:' has lower precedence than '&'; '&' will be evaluated first [-Werror,-Wbitwise-conditional-parentheses] ? (delta(&tm, &lotm) == t - lot which is exactly what the code intendes. Instead of adding a warning suppress, place parentheses around the '&' expression. Diff: --- timezone/zdump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/timezone/zdump.c b/timezone/zdump.c index b532fe3eae..bfc01faffa 100644 --- a/timezone/zdump.c +++ b/timezone/zdump.c @@ -665,7 +665,7 @@ hunt(timezone_t tz, char *name, time_t lot, time_t hit) else if (t >= hit) --t; tm_ok = my_localtime_rz(tz, &t, &tm) != NULL; - if (lotm_ok & tm_ok + if ((lotm_ok & tm_ok) ? (delta(&tm, &lotm) == t - lot && tm.tm_isdst == lotm.tm_isdst && strcmp(abbr(&tm), ab) == 0)