Re: [PATCH 0/1] Enable -Wshadow=local

[Warner Losh]

On Wed, Oct 25, 2023, 11:31 PM Markus Armbruster <armbru@redhat.com> wrote:

Requires Brian's pull request and two patches from Thomas to compile:

    [PULL 0/2] hex queue - GETPC() fixes, shadowing fixes
    [PATCH v2] block/snapshot: Fix compiler warning with -Wshadow=local
    [PATCH v2] migration/ram: Fix compilation with -Wshadow=local

Stefan, the PR was posted a week ago; anything blocking it?

Warner, I believe not waiting for your cleanup of bsd-user is fine.
Please holler if it isn't.

If it's not enabled by default for Clang, then sure. It's only one small change at this point, but i was ill for a few weeks (much longer than i thought I'd be) and am still catching up.

Warner

 <20231019021733.2258592-1-bcain@quicinc.com" target="_blank" rel="noreferrer">20231019021733.2258592-1-bcain@quicinc.com>

Based-on: <20231023175038.111607-1-thuth@redhat.com" target="_blank" rel="noreferrer">20231023175038.111607-1-thuth@redhat.com>
Based-on: <20231024092220.55305-1-thuth@redhat.com" target="_blank" rel="noreferrer">20231024092220.55305-1-thuth@redhat.com>

Markus Armbruster (1):
  meson: Enable -Wshadow=local

 meson.build | 1 +
 1 file changed, 1 insertion(+)

--
2.41.0