From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1791) id 92ED4385842C; Wed, 7 Feb 2024 14:06:17 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 92ED4385842C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1707314777; bh=DYdbV+7WBfuxVe8ut4c+9QenjEIhJ5SsS6AYDmerW4E=; h=From:To:Subject:Date:From; b=msQW2ktw19TiR9SkPh8jUoVhseKa8Hj/ji1qucrl/plW4s4uPX+bmtsbhH9NaQbEu QS0RBofTQXyvx7PAzIdCxyyCnskPqSYnRj/Rir7gvi51qknBadEiFTUXB2ctr8/9Ii CEyc34yTIZU/Q6w2sP5UhSEJUIHhFvYKuFREaub4= 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: f94bfb88617e8b405f864ad7a78041605a3bc0d4 X-Git-Newrev: 020d69d12deb8f88c8d4a1709569e43dd01345b2 Message-Id: <20240207140617.92ED4385842C@sourceware.org> Date: Wed, 7 Feb 2024 14:06:17 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=020d69d12deb8f88c8d4a1709569e43dd01345b2 commit 020d69d12deb8f88c8d4a1709569e43dd01345b2 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)