Skip to content

Fix warning message C4100 #163

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.

Sign up for GitHub

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

Jump to bottom
Closed
utilForever wants to merge 1 commit into from
Closed

Fix warning message C4100 #163

utilForever wants to merge 1 commit into from

Conversation

@utilForever
Copy link

@utilForever utilForever commented Mar 27, 2021

  • warning C4100: 'bgColor': unreferenced formal parameter

@utilForever
Fix warning message C4100
8514acf 
- warning C4100: 'bgColor': unreferenced formal parameter
@eliasdaler
Copy link
Contributor

eliasdaler commented Mar 27, 2021

Hello
It doesn't seem like it's needed after #162. Can you please test on the latest master? The C4100 should be gone by now - it was an error because bgColor was not passed into ImGui::ImageButton, but it's now passed, so it's not unreferenced.

@utilForever
Copy link
Author

utilForever commented Mar 28, 2021

Oh, I'm so sorry. I tested on the latest master and confirmed the build is passed. 😂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@utilForever @eliasdaler