From nobody Sat Oct 25 20:28:05 2025 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4cvBDM57fMz6DSRC for ; Sat, 25 Oct 2025 20:28:19 +0000 (UTC) (envelope-from rlibby@gmail.com) Received: from mail-yw1-f173.google.com (mail-yw1-f173.google.com [209.85.128.173]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "WR4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4cvBDL3FY3z3ZDT for ; Sat, 25 Oct 2025 20:28:18 +0000 (UTC) (envelope-from rlibby@gmail.com) Authentication-Results: mx1.freebsd.org; dkim=none; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=freebsd.org (policy=none); spf=pass (mx1.freebsd.org: domain of rlibby@gmail.com designates 209.85.128.173 as permitted sender) smtp.mailfrom=rlibby@gmail.com Received: by mail-yw1-f173.google.com with SMTP id 00721157ae682-784966ad073so41033087b3.1 for ; Sat, 25 Oct 2025 13:28:18 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1761424097; x=1762028897; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=7xmCQfITdSRooTSdhPD6juZeD60m4HtCllE3eY7sN3E=; b=vnmyIl/h1bjWKf/Srkk1UfHjc098rqqJ183iIyjzOmTH3mw+AMs2uP4p4w65YR57YA tlrCcpuAoDeVl9eTgiRFqAera/WPKVuTzgS0HD62hd6yyqDc7HPBf6iCIPF2H5i++JRT SHMTfkNMBe1js3fwDYyBiQKxfYzC/fbbKUtZM4cozzVsRb9r+Pa9cNz48lXhDP07Efo3 ZU+NGM++NsWECpBZsvFHNREz3vPEkq9VBt7Ll4G/MI1TI1izTeLKXEAX/7PFMUWyalI+ UShzTu/0mgoKNUHKZ8jzAck7jL1NSt+5N9v/TrZ5w+K8dym68gmHzgq2TmEhw1+5lP0l dfmg== X-Forwarded-Encrypted: i=1; AJvYcCXeNWAHQovOLN8PdIzP6UVldbwTgV3AZs1r1QXRr8f75m4oV1kFFe4Sf8INH2qxTYOMpUH3MddH/+i1ilc9gq50sRGXDw==@freebsd.org X-Gm-Message-State: AOJu0YyK6NLech2Im4wfqAwDgk1slSUeTzXxzVF4foucV156E0r/vMVP 1d4AUYPBlpA6Ehdodv3J5hZx/Jaxd/M3N8Mg55Ir7oGoF7A5NHLlXoR+MsgF+A== X-Gm-Gg: ASbGncvzCz4EqolFXqn2Y2z3O+Z9zIhLDI3/SXEMrCkZ1y3zV16ki7oN2ZSKZVE2JBg EsC6le9QLC9b2s0jeqX0iIbpubsk0NIFGS2ifUq/RY7p5MiCCaBXt97xM11u35SzGAw7USn7Evh tDddR57CeBwVzbPabFmJEo+r5kl12fJKZ0ZMGg+weZBHpv7T38c84NCY4YuecYoc+vqgz3ZKo5s QSb7whwFKy8T40Z5DDnvD7qEjab4iHja6Y5D39CxRdZbpX/KyI39J/0xl4PjrhCthXf8P1QcHWS K4JVqvfQbzUcsdAGJhIzoY+BlymhVT1FyMTBDD6YDOEAgKDb+D8gJt/CG5YB1pP0BC4NZRjJ/NM J/OENLYq/rWh1wMwjp9ybsVW5+rGe5h0xAppNiClGcskwrTcBjo/VfOIf6159APMBRML/HXjSL1 l6KUn57IJxL2fhM4wi0t/jsYFQAN+r/pdsAf8zzu8= X-Google-Smtp-Source: AGHT+IE18ey17MX2yVmJRUcDzv4SiHc7E964OgIOTyZfR6NSTSBf63Poqn39Vgp81RAysyfBOunpsQ== X-Received: by 2002:a05:690e:12c7:b0:63e:e04:fc31 with SMTP id 956f58d0204a3-63e161c7871mr22646818d50.43.1761424097017; Sat, 25 Oct 2025 13:28:17 -0700 (PDT) Received: from mail-yw1-f172.google.com (mail-yw1-f172.google.com. [209.85.128.172]) by smtp.gmail.com with ESMTPSA id 00721157ae682-785ed196a0dsm7314487b3.27.2025.10.25.13.28.16 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 25 Oct 2025 13:28:16 -0700 (PDT) Received: by mail-yw1-f172.google.com with SMTP id 00721157ae682-784966ad073so41032987b3.1 for ; Sat, 25 Oct 2025 13:28:16 -0700 (PDT) X-Forwarded-Encrypted: i=1; AJvYcCUw4IosPowpyLi4EtPiAAVM4hk65NsTndWKAXlSaS8SoBe45kTxxJKGtPB8IGYYsFsJTzAAYNMHG/066ssu5IlUT64s3g==@freebsd.org X-Received: by 2002:a05:690c:385:b0:785:4803:5e2b with SMTP id 00721157ae682-785480363fcmr159586017b3.20.1761424096185; Sat, 25 Oct 2025 13:28:16 -0700 (PDT) List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-main@freebsd.org Sender: owner-dev-commits-src-main@FreeBSD.org MIME-Version: 1.0 References: <202510201832.59KIWCG0006624@gitrepo.freebsd.org> In-Reply-To: <202510201832.59KIWCG0006624@gitrepo.freebsd.org> From: Ryan Libby Date: Sat, 25 Oct 2025 13:28:05 -0700 X-Gmail-Original-Message-ID: X-Gm-Features: AS18NWC_bNRGAAreIS_VhM_9PnPJamxfoMxBYzoI-mimPHVTTUSJ-qgi21BfEGg Message-ID: Subject: Re: git: e1aeb58cbbc3 - main - inet_net_test: Use int to hold expected return values from inet_net_pton To: John Baldwin Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Spamd-Bar: - X-Spamd-Result: default: False [-1.79 / 15.00]; NEURAL_HAM_SHORT(-1.00)[-0.999]; NEURAL_HAM_MEDIUM(-0.80)[-0.798]; FORGED_SENDER(0.30)[rlibby@freebsd.org,rlibby@gmail.com]; R_SPF_ALLOW(-0.20)[+ip4:209.85.128.0/17:c]; DMARC_POLICY_SOFTFAIL(0.10)[freebsd.org : SPF not aligned (relaxed), No valid DKIM,none]; MIME_GOOD(-0.10)[text/plain]; NEURAL_HAM_LONG(-0.10)[-0.095]; RCPT_COUNT_THREE(0.00)[4]; RCVD_TLS_LAST(0.00)[]; RCVD_COUNT_THREE(0.00)[3]; ARC_NA(0.00)[]; MISSING_XM_UA(0.00)[]; TO_DN_SOME(0.00)[]; MIME_TRACE(0.00)[0:+]; FROM_HAS_DN(0.00)[]; ASN(0.00)[asn:15169, ipnet:209.85.128.0/17, country:US]; FREEMAIL_ENVFROM(0.00)[gmail.com]; PREVIOUSLY_DELIVERED(0.00)[dev-commits-src-main@freebsd.org]; TO_MATCH_ENVRCPT_SOME(0.00)[]; FROM_NEQ_ENVFROM(0.00)[rlibby@freebsd.org,rlibby@gmail.com]; FREEFALL_USER(0.00)[rlibby]; MLMMJ_DEST(0.00)[dev-commits-src-main@freebsd.org]; RWL_MAILSPIKE_POSSIBLE(0.00)[209.85.128.173:from]; R_DKIM_NA(0.00)[]; RCVD_VIA_SMTP_AUTH(0.00)[]; RCVD_IN_DNSWL_NONE(0.00)[209.85.128.172:received,209.85.128.173:from] X-Rspamd-Queue-Id: 4cvBDL3FY3z3ZDT On Mon, Oct 20, 2025 at 11:32=E2=80=AFAM John Baldwin wro= te: > > The branch main has been updated by jhb: > > URL: https://cgit.FreeBSD.org/src/commit/?id=3De1aeb58cbbc3839db93ec38ce4= 91b7b9383d5649 > > commit e1aeb58cbbc3839db93ec38ce491b7b9383d5649 > Author: John Baldwin > AuthorDate: 2025-10-20 18:31:41 +0000 > Commit: John Baldwin > CommitDate: 2025-10-20 18:31:41 +0000 > > inet_net_test: Use int to hold expected return values from inet_net_p= ton > > GCC warns about the sign mismatch in comparisons: > > lib/libc/tests/net/inet_net_test.cc: In member function 'virtual void= {anonymous}::atfu_tc_inet_net_inet4::body() const': > lib/libc/tests/net/inet_net_test.cc:86:17: error: comparison of integ= er expressions of different signedness: 'int' and 'const unsigned int' [-We= rror=3Dsign-compare] > 86 | ATF_REQUIRE_EQ(bits, addr.bits); > | ^~~~~~~~~~~~~~ > lib/libc/tests/net/inet_net_test.cc: In member function 'virtual void= {anonymous}::atfu_tc_inet_net_inet6::body() const': > lib/libc/tests/net/inet_net_test.cc:205:17: error: comparison of inte= ger expressions of different signedness: 'int' and 'const unsigned int' [-W= error=3Dsign-compare] > 205 | ATF_REQUIRE_EQ(bits, addr.bits); > | ^~~~~~~~~~~~~~ > > Fixes: 8f4a0d2f7b96 ("libc: Import OpenBSD's inet_net_{ntop,= pton}") > --- > lib/libc/tests/net/inet_net_test.cc | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/lib/libc/tests/net/inet_net_test.cc b/lib/libc/tests/net/ine= t_net_test.cc > index 4ecf5a3de492..c4411373165e 100644 > --- a/lib/libc/tests/net/inet_net_test.cc > +++ b/lib/libc/tests/net/inet_net_test.cc > @@ -50,7 +50,7 @@ ATF_TEST_CASE_BODY(inet_net_inet4) > > struct test_addr { > std::string input; > - unsigned bits; > + int bits; > std::string output; > }; > > @@ -134,7 +134,7 @@ ATF_TEST_CASE_BODY(inet_net_inet6) > > struct test_addr { > std::string input; > - unsigned bits; > + int bits; > std::string output; > }; > > Could you MFC e1aeb58cbbc3 and aa358ce3ca8e to stable/15 to fix the gcc build there, or would you mind if I did? Thanks, Ryan