Fix unused-parameter warning from compillation - Githubissues

obsproject / obs-browser

CEF-based OBS Studio browser plugin

GNU General Public License v2.0

776 stars 220 forks source link

Fix unused-parameter warning from compillation #335

Closed tytan652 closed 2 years ago

tytan652 commented 2 years ago

Description

Related to https://github.com/obsproject/obs-studio/pull/5653

This PR is meant to fix some compilation warnings, ignoring FFmpeg, AJA NT2 deprecation and FTL related ones:

obs-browser emits a unused-parameter warning if built with the new CEF

Motivation and Context

Less there is warning, the better is. No ? Yes, I don't like warnings so I tried to fix what I can fix.

How Has This Been Tested?

I just build this branch on Arch Linux with the new CEF and no warning from obs-browser were emitted.

Types of changes

Code cleanup (non-breaking change which makes code smaller or more readable)

Checklist:

[x] My code has been run through clang-format.
[x] I have read the contributing document.
[x] My code is not on the master branch.
[x] The code has been tested.
[x] All commit messages are properly formatted and commits squashed where appropriate.
[x] I have included updates to all appropriate documentation.