git: 78bbe9fc3e35 - main - pmc: Use distinct names for arguments to the pmc_config constructor

John Baldwin <[email protected]>
Newsgroups gmane.os.freebsd.devel.cvs.src
Message-ID <6a876751.30a48.559158c9__19883.1792267569$1787258737$gmane$org@gitrepo.freebsd.org>
The branch main has been updated by jhb:

URL: https://cgit.FreeBSD.org/src/commit/?id=78bbe9fc3e3529731df07e29464f103f96f6ebed

commit 78bbe9fc3e3529731df07e29464f103f96f6ebed
Author:     John Baldwin <[email protected]>
AuthorDate: 2026-08-20 19:49:36 +0000
Commit:     John Baldwin <[email protected]>
CommitDate: 2026-08-20 19:49:36 +0000

    pmc: Use distinct names for arguments to the pmc_config constructor
    
    This pacifies shadow warnings from GCC:
    
    usr.sbin/pmc/cmd_pmc_record.cc: In constructor 'pmc_config::pmc_config(const std::__1::string&, uint64_t, cpuset_t)':
    usr.sbin/pmc/cmd_pmc_record.cc:96:71: error: declaration of 'cpumask' shadows a member of 'pmc_config' [-Werror=shadow]
       96 |         pmc_config(const std::string &event, uint64_t count, cpuset_t cpumask)
          |                                                              ~~~~~~~~~^~~~~~~
    usr.sbin/pmc/cmd_pmc_record.cc:90:25: note: shadowed declaration is here
       90 |         cpuset_t        cpumask;
          |                         ^~~~~~~
    usr.sbin/pmc/cmd_pmc_record.cc:96:55: error: declaration of 'count' shadows a member of 'pmc_config' [-Werror=shadow]
       96 |         pmc_config(const std::string &event, uint64_t count, cpuset_t cpumask)
          |                                              ~~~~~~~~~^~~~~
    usr.sbin/pmc/cmd_pmc_record.cc:89:25: note: shadowed declaration is here
       89 |         uint64_t        count;
          |                         ^~~~~
    usr.sbin/pmc/cmd_pmc_record.cc:96:39: error: declaration of 'event' shadows a member of 'pmc_config' [-Werror=shadow]
       96 |         pmc_config(const std::string &event, uint64_t count, cpuset_t cpumask)
          |                    ~~~~~~~~~~~~~~~~~~~^~~~~
    usr.sbin/pmc/cmd_pmc_record.cc:88:25: note: shadowed declaration is here
       88 |         std::string     event;
          |                         ^~~~~
    
    Reported by:    GCC 15
    Fixes:          a79a051e7d16 ("pmc: pmc record command")
---
 usr.sbin/pmc/cmd_pmc_record.cc | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/usr.sbin/pmc/cmd_pmc_record.cc b/usr.sbin/pmc/cmd_pmc_record.cc
index a8373b330705..cff0db7fc9c7 100644
--- a/usr.sbin/pmc/cmd_pmc_record.cc
+++ b/usr.sbin/pmc/cmd_pmc_record.cc
@@ -93,10 +93,11 @@ public:
 	{
 		CPU_ZERO(&cpumask);
 	}
-	pmc_config(const std::string &event, uint64_t count, cpuset_t cpumask)
-	    : event(event), count(count), ids()
+	pmc_config(const std::string &_event, uint64_t _count,
+	    cpuset_t _cpumask)
+	    : event(_event), count(_count), ids()
 	{
-		CPU_COPY(&cpumask, &this->cpumask);
+		CPU_COPY(&_cpumask, &cpumask);
 	}
 	/*
 	 * Read the capabilities and fills in the cpumask for a given event
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.