Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

-Wshadow or -Wno-shadow? #132

Open
ipatix opened this issue Nov 23, 2024 · 2 comments
Open

-Wshadow or -Wno-shadow? #132

ipatix opened this issue Nov 23, 2024 · 2 comments

Comments

@ipatix
Copy link

ipatix commented Nov 23, 2024

Operating System Info

Ubuntu 24.04

Other OS

No response

OBS Studio Version

31.0.0-rc1

OBS Studio Version (Other)

No response

OBS Studio Log URL

N/A

OBS Studio Crash Log URL

No response

Expected Behavior

see below

Current Behavior

see below

Steps to Reproduce

see below

Anything else we should know?

I'm sorry, but it seems once again the issue template really does not apply to my case, anyway:

I discovered this while compiling my own plugin that errors are raised due to -Wshadow. Now, this may seem perfectly fine, but this led me to the place where this warning is enabled, and it leads to here:

Now, what I find confusing about this is that just a few lines above -Wno-shadow is specified:

This question goes to you, the maintainers: So is this warning supposed to be enabled or disabled?

@Fenrirthviti
Copy link
Member

As a note, the issue template doesn't apply to you because you shouldn't be submitting issues for support questions.

I'll leave this open for someone else to comment on, but in the future, we recommend our forums or Discord for assistance.

@ipatix
Copy link
Author

ipatix commented Nov 24, 2024

Well, at least to me this looks like a bug. However, since I don't know the intention behind the -Wshadow I cannot comment in regards to the "expected" and "actual" behavior.

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

No branches or pull requests

2 participants