freak82 opened a new pull request, #11524: URL: https://github.com/apache/trafficserver/pull/11524
- Use `/* name ATS_UNUSED */` for fully unused parameters in C++ files - Use `[[maybe_unused]]` for conditionally unused parameters in C++ files - Use `name ATS_UNUSED` for unused parameters in C files I wasn't sure if the use `(void)param` inside the functions or `ATS_UNUSED` for the C files which are compiled as such. There can't be used `/* name ATS_UNUSED */` because this requires C2X build. I decided to use `ATS_UNUSED` because it's closer to the way the C++ files are fixed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@trafficserver.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org