Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 3 Apr 2024 18:53:17 +0100
From:      Jessica Clarke <jrtc27@freebsd.org>
To:        =?utf-8?Q?Bojan_Novkovi=C4=87?= <bnovkov@FreeBSD.org>
Cc:        "src-committers@freebsd.org" <src-committers@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>
Subject:   Re: git: 872c4402af13 - main - ddb: Don't throw away qualifier when pretty-printing unnamed pointers
Message-ID:  <7D6B67DF-D648-49F2-A03B-D7F9A509CA79@freebsd.org>
In-Reply-To: <202404031618.433GIQUx096602@gitrepo.freebsd.org>
References:  <202404031618.433GIQUx096602@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 3 Apr 2024, at 17:18, Bojan Novkovi=C4=87 <bnovkov@FreeBSD.org> =
wrote:
>=20
> The branch main has been updated by bnovkov:
>=20
> URL: =
https://cgit.FreeBSD.org/src/commit/?id=3D872c4402af1325ed156d7f5ee2252dd3=
6b016b5c
>=20
> commit 872c4402af1325ed156d7f5ee2252dd36b016b5c
> Author:     Bojan Novkovi=C4=87 <bnovkov@FreeBSD.org>
> AuthorDate: 2024-04-03 15:55:13 +0000
> Commit:     Bojan Novkovi=C4=87 <bnovkov@FreeBSD.org>
> CommitDate: 2024-04-03 16:17:54 +0000
>=20
>    ddb: Don't throw away qualifier when pretty-printing unnamed =
pointers
>=20
>    Reported by:    jrtc27
>    Fixes:  c21bc6f ("ddb: Add CTF-based pretty printing")
>    Approved by:    markj (mentor)
> ---
> sys/ddb/db_pprint.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>=20
> diff --git a/sys/ddb/db_pprint.c b/sys/ddb/db_pprint.c
> index b4116372cf65..8e6759a03220 100644
> --- a/sys/ddb/db_pprint.c
> +++ b/sys/ddb/db_pprint.c
> @@ -276,7 +276,7 @@ db_pprint_ptr(db_addr_t addr, struct ctf_type_v3 =
*type, u_int depth)
> if (name !=3D NULL)
> db_printf("(%s%s *) 0x%lx", qual, name, (long)val);
> else
> - db_printf("0x%lx", (long)val);
> + db_printf("(%s *) 0x%lx", qual, (long)val);

At this point, why not just:

db_printf("(%s%s *) 0x%lx", qual, name !=3D NULL ? name : "", =
(long)val);

Jess




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7D6B67DF-D648-49F2-A03B-D7F9A509CA79>