Skip to content

Suppress unused var warnings#153

Merged
andersson merged 2 commits intolinux-msm:masterfrom
igoropaniuk:unused_params_warnings
Nov 13, 2025
Merged

Suppress unused var warnings#153
andersson merged 2 commits intolinux-msm:masterfrom
igoropaniuk:unused_params_warnings

Conversation

@igoropaniuk
Copy link
Contributor

  1. Introduce __unused macro to prevent warnings when a function parameter
    might not be used.
  2. Use __unused attribute to suppress compile warnings for callback
    implementations, where some parameters aren't used

Introduce __unused macro to prevent warnings when a function parameter
might not be used.

Signed-off-by: Igor Opaniuk <igor.opaniuk@oss.qualcomm.com>
Use __unused attribute to suppress compile warnings for callback
implementations, where some parameters aren't used. For example:

warning: unused parameter ‘qdl’ [-Wunused-parameter]
./firehose.c:257:80: warning: unused parameter ‘rawmode’ [-Wunused-parameter]
  257 | static int firehose_configure_response_parser(xmlNode *node...

Signed-off-by: Igor Opaniuk <igor.opaniuk@oss.qualcomm.com>
@igoropaniuk igoropaniuk force-pushed the unused_params_warnings branch from 5e7075e to 904e405 Compare November 13, 2025 10:49
@igoropaniuk igoropaniuk changed the title Address unused warnings for some callback implementations Suppress compile warnings for callback implementations, where some parameters aren't used Nov 13, 2025
@igoropaniuk igoropaniuk changed the title Suppress compile warnings for callback implementations, where some parameters aren't used Suppress unused var warnings Nov 13, 2025
@andersson
Copy link
Collaborator

Please start your commit messages with a clear "problem statement", ensure that the current or future reader understand why the particular change is being made. Then continue with the explanation of the change, as you do today.

@andersson andersson merged commit a79a572 into linux-msm:master Nov 13, 2025
13 checks passed
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.

2 participants