Skip to content

Remove shadowing by making use of more explicit names#248

Merged
milesburton merged 1 commit intomilesburton:masterfrom
cjoach:no-shadow
Oct 25, 2023
Merged

Remove shadowing by making use of more explicit names#248
milesburton merged 1 commit intomilesburton:masterfrom
cjoach:no-shadow

Conversation

@cjoach
Copy link
Copy Markdown
Contributor

@cjoach cjoach commented Oct 25, 2023

The name bitResolution is used in different contexts in the library and sometimes the local name shadows the class attribute. This PR modifies the names to be more explicit and remove the -Wshadow warnings.

@RobTillaart
Copy link
Copy Markdown
Contributor

@cjoach
Which platform/ compiler gives these warning?

@milesburton
Copy link
Copy Markdown
Owner

Hmm interesting PR @cjoach. Looks like a decent change. Any issues @RobTillaart ?

@RobTillaart
Copy link
Copy Markdown
Contributor

@milesburton
I checked the code and there are still a handful occurrences of bitResolution.
However these are all parameters of functions and could be either device- or globalBitResolution.
So they are all renamed consistently ==> no problemo with this PR.
LGTM.

@milesburton
Copy link
Copy Markdown
Owner

As always. thanks @RobTillaart . Again thanks for your contribution @cjoach

@milesburton milesburton merged commit 4924e2b into milesburton:master Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants