Date: Sun, 10 Jan 2021 13:59:02 +0100 From: Dimitry Andric <dim@FreeBSD.org> To: Cy Schubert <Cy.Schubert@cschubert.com> Cc: Chuck Tuffli <chuck@freebsd.org>, Jessica Clarke <jrtc27@freebsd.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@freebsd.org>, "dev-commits-src-main@freebsd.org" <dev-commits-src-main@freebsd.org>, "src-committers@freebsd.org" <src-committers@freebsd.org> Subject: Re: git: 6733401935f8 - main - nvmecontrol: add device self-test op and log page Message-ID: <2B913109-27A8-4ABF-8F64-48CE78B76759@FreeBSD.org> In-Reply-To: <202101082253.108MrrUL003251@slippy.cwsent.com> References: <202101081729.108HTG7j066884@gitrepo.freebsd.org> <74314BD0-A0F8-481A-93A0-28FB83CB2717@freebsd.org> <202101081920.108JKUd2003299@slippy.cwsent.com> <202101081927.108JRKxR084004@slippy.cwsent.com> <CAKAYmMLD5F7C_kBSVJ6vXgGmBFDK_J4053WeMSOJK07P-VQ_8g@mail.gmail.com> <202101082253.108MrrUL003251@slippy.cwsent.com>
next in thread | previous in thread | raw e-mail | index | archive | help
--Apple-Mail=_B662E377-B892-42B0-A9EE-915CA5C94CBD Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=us-ascii On 8 Jan 2021, at 23:53, Cy Schubert <Cy.Schubert@cschubert.com> wrote: >=20 > In message = <CAKAYmMLD5F7C_kBSVJ6vXgGmBFDK_J4053WeMSOJK07P-VQ_8g@mail.gmail.c > om> > , Chuck Tuffli writes: >> --000000000000c9481f05b86a0c60 >> Content-Type: text/plain; charset=3D"UTF-8" >>=20 >> On Fri, Jan 8, 2021 at 11:27 AM Cy Schubert = <Cy.Schubert@cschubert.com> >> wrote: >>=20 >>> In message <202101081920.108JKUd2003299@slippy.cwsent.com>, Cy = Schubert >>> writes: >>>> In message <74314BD0-A0F8-481A-93A0-28FB83CB2717@freebsd.org>, = Jessica >>>> Clarke w >>>> rites: >>>=20 >> ... >>=20 >>>>> This broke powerpc and powerpc64: >>=20 >>=20 >> Sorry about that. I'll fix this in a bit. >>=20 >> --chuck >=20 > Looks like dim@ already did. >=20 > The time before last I fixed one of these it was pointed out to me > privately that when we use %jd that we should also cast to intmax_t or > uintmax_t, and that jhb@ told him. This was to future-proof, assuming = we'd > have 128-bit processors one day. That is certainly valid for e.g. time_t or other types with a varying width, or where you cannot predict the exact width. But for printing uint64_t and friends, where the width *is* exactly specified, I would rather just use the standards-defined PRIu64 and so on. Casting will obscure any type mismatches between the printf format strings and the actual arguments. -Dimitry --Apple-Mail=_B662E377-B892-42B0-A9EE-915CA5C94CBD Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.2 iF0EARECAB0WIQR6tGLSzjX8bUI5T82wXqMKLiCWowUCX/r6FgAKCRCwXqMKLiCW o+J+AKCeYyD2580Oi4rzzHPSGP87XkiNcACfRsz+iNOqMwrRN1jtksK75qLGIWA= =VpgJ -----END PGP SIGNATURE----- --Apple-Mail=_B662E377-B892-42B0-A9EE-915CA5C94CBD--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?2B913109-27A8-4ABF-8F64-48CE78B76759>