René's URL Explorer Experiment


Title: fail `run-clang-tidy` in case of compiler warnings / fixed Clang warnings / cleanups by firewave · Pull Request #4036 · cppcheck-opensource/cppcheck · GitHub

Open Graph Title: fail `run-clang-tidy` in case of compiler warnings / fixed Clang warnings / cleanups by firewave · Pull Request #4036 · cppcheck-opensource/cppcheck

X Title: fail `run-clang-tidy` in case of compiler warnings / fixed Clang warnings / cleanups by firewave · Pull Request #4036 · cppcheck-opensource/cppcheck

Description: static analysis of C/C++ code. Contribute to cppcheck-opensource/cppcheck development by creating an account on GitHub.

Open Graph Description: clang-tidy will also report compiler warnings which we were currently suppressing since we treat all warnings as errors. We should make sure our code is free of warnings. So we should fail the buil...

X Description: clang-tidy will also report compiler warnings which we were currently suppressing since we treat all warnings as errors. We should make sure our code is free of warnings. So we should fail the buil...

Opengraph URL: https://github.com/cppcheck-opensource/cppcheck/pull/4036

X: @github

direct link

Domain: github.com

route-pattern/:user_id/:repository/pull/:id/commits(.:format)
route-controllerpull_requests
route-actioncommits
fetch-noncev2:d91e366c-c6db-6f0c-fe9e-14a7111c352a
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idD062:FC5F:247BFE:357098:6A4E0002
html-safe-nonce94c9711f4a3ad944404ce55eb6bf7748b77934a791bcbafcdc742cdc9a8e4f6e
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJEMDYyOkZDNUY6MjQ3QkZFOjM1NzA5ODo2QTRFMDAwMiIsInZpc2l0b3JfaWQiOiI4Mjc0NzgzMzQ1NDQwNTg3Nzc4IiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0=
visitor-hmac105ea1a6c7a1430b5c587071d4a761fa97aa84260d4b61651464a86ab27987d1
hovercard-subject-tagpull_request:915003682
github-keyboard-shortcutsrepository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///pull_requests/show/commits
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits
twitter:imagehttps://opengraph.githubassets.com/9e602cac44fe46b1e585a9f1b3052ebb185169727d1985c3349e7bfcc72341e2/cppcheck-opensource/cppcheck/pull/4036
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/9e602cac44fe46b1e585a9f1b3052ebb185169727d1985c3349e7bfcc72341e2/cppcheck-opensource/cppcheck/pull/4036
og:image:altclang-tidy will also report compiler warnings which we were currently suppressing since we treat all warnings as errors. We should make sure our code is free of warnings. So we should fail the buil...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernamefirewave
hostnamegithub.com
expected-hostnamegithub.com
Nonedf0492960db29b4938cb72070351d6b1d0c6c0767b27ceb8394bbf4fcc0223c6
turbo-cache-controlno-cache
go-importgithub.com/cppcheck-opensource/cppcheck git https://github.com/cppcheck-opensource/cppcheck.git
octolytics-dimension-user_id278097703
octolytics-dimension-user_logincppcheck-opensource
octolytics-dimension-repository_id143131
octolytics-dimension-repository_nwocppcheck-opensource/cppcheck
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id143131
octolytics-dimension-repository_network_root_nwocppcheck-opensource/cppcheck
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release52bde38e24398476f8eb1e0760c81346c6a00812
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcppcheck-opensource%2Fcppcheck%2Fpull%2F4036%2Fcommits
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub Copilot appDirect agents from issue to mergehttps://github.com/features/ai/github-app
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
View all resourceshttps://github.com/resources
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
GitHub Starshttps://stars.github.com
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcppcheck-opensource%2Fcppcheck%2Fpull%2F4036%2Fcommits
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fpull_requests%2Fshow%2Fcommits&source=header-repo&source_repo=cppcheck-opensource%2Fcppcheck
Reloadhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits
Reloadhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits
Reloadhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits
Please reload this pagehttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits
cppcheck-opensource https://github.com/cppcheck-opensource
cppcheckhttps://github.com/cppcheck-opensource/cppcheck
Notifications https://github.com/login?return_to=%2Fcppcheck-opensource%2Fcppcheck
Fork 1.6k https://github.com/login?return_to=%2Fcppcheck-opensource%2Fcppcheck
Star 6.7k https://github.com/login?return_to=%2Fcppcheck-opensource%2Fcppcheck
Code https://github.com/cppcheck-opensource/cppcheck
Pull requests 204 https://github.com/cppcheck-opensource/cppcheck/pulls
Actions https://github.com/cppcheck-opensource/cppcheck/actions
Security and quality 0 https://github.com/cppcheck-opensource/cppcheck/security
Insights https://github.com/cppcheck-opensource/cppcheck/pulse
Code https://github.com/cppcheck-opensource/cppcheck
Pull requests https://github.com/cppcheck-opensource/cppcheck/pulls
Actions https://github.com/cppcheck-opensource/cppcheck/actions
Security and quality https://github.com/cppcheck-opensource/cppcheck/security
Insights https://github.com/cppcheck-opensource/cppcheck/pulse
danmarhttps://github.com/danmar
cppcheck-opensource:mainhttps://github.com/cppcheck-opensource/cppcheck/tree/main
firewave:tidy-warnhttps://github.com/firewave/cppcheck/tree/tidy-warn
Conversationhttps://github.com/cppcheck-opensource/cppcheck/pull/4036
Commits13 (13)https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits
Checkshttps://github.com/cppcheck-opensource/cppcheck/pull/4036/checks
Files changedhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/files
fail run-clang-tidy in case of compiler warnings / fixed Clang warnings / cleanupshttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits#top
danmarhttps://github.com/danmar
cppcheck-opensource:mainhttps://github.com/cppcheck-opensource/cppcheck/tree/main
firewave:tidy-warnhttps://github.com/firewave/cppcheck/tree/tidy-warn
properly suppress clang-tidy warning in importproject.cpphttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/77fb49bee25f3b7aad0c87a4ba951715cd9b7228
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
77fb49bhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/77fb49bee25f3b7aad0c87a4ba951715cd9b7228
https://github.com/cppcheck-opensource/cppcheck/tree/77fb49bee25f3b7aad0c87a4ba951715cd9b7228
clang_tidy.cmake: fail if any compiler warning is detected - saves us an additional CI stephttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/ee7e2465d459aac513f627587591a5d4b3d57779
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
ee7e246https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/ee7e2465d459aac513f627587591a5d4b3d57779
https://github.com/cppcheck-opensource/cppcheck/tree/ee7e2465d459aac513f627587591a5d4b3d57779
suppressions.h: fixed -Wdocumentation warninghttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/9e584d20002b3abff2de227ac1853654b1171015
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
9e584d2https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/9e584d20002b3abff2de227ac1853654b1171015
https://github.com/cppcheck-opensource/cppcheck/tree/9e584d20002b3abff2de227ac1853654b1171015
fixed -Wshadow warning for testcppchecklibrarydata.cpp::loadCfgFile()https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/2c12f183378cffc59a16d9950471b2549ec1ed7c
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
2c12f18https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/2c12f183378cffc59a16d9950471b2549ec1ed7c
https://github.com/cppcheck-opensource/cppcheck/tree/2c12f183378cffc59a16d9950471b2549ec1ed7c
config.h: added `UNUSED` macrohttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/e88de112aca9e91763f5a02e1fc6b3777e6da8da
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
e88de11https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/e88de112aca9e91763f5a02e1fc6b3777e6da8da
https://github.com/cppcheck-opensource/cppcheck/tree/e88de112aca9e91763f5a02e1fc6b3777e6da8da
resultstree.cpp: removed unused functionhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/ebc792e40fcd342f87a47aa4a52bd8d6c2ea9285
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
ebc792ehttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/ebc792e40fcd342f87a47aa4a52bd8d6c2ea9285
https://github.com/cppcheck-opensource/cppcheck/tree/ebc792e40fcd342f87a47aa4a52bd8d6c2ea9285
translationhandler.cpp: avoid -Wunused-function warninghttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/35cd258fcb8edcc4e558028a062cb700930e6791
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
35cd258https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/35cd258fcb8edcc4e558028a062cb700930e6791
https://github.com/cppcheck-opensource/cppcheck/tree/35cd258fcb8edcc4e558028a062cb700930e6791
resultstree.cpp: avoid -Wdeprecated-declarations warninghttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/afd595bcfe0db1a44f972c94016271cda95675c4
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
afd595bhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/afd595bcfe0db1a44f972c94016271cda95675c4
https://github.com/cppcheck-opensource/cppcheck/tree/afd595bcfe0db1a44f972c94016271cda95675c4
removed some unnecessary `__cplusplus` preprocessor checkshttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/81efa0ffb61bc9da2d08eaa3bde802578a639a4c
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
81efa0fhttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/81efa0ffb61bc9da2d08eaa3bde802578a639a4c
https://github.com/cppcheck-opensource/cppcheck/tree/81efa0ffb61bc9da2d08eaa3bde802578a639a4c
config.h: made `SUPPRESS_WARNING` usable by any codehttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/a77ef38eca39bd55b2cba982f19812f0920728d2
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
a77ef38https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/a77ef38eca39bd55b2cba982f19812f0920728d2
https://github.com/cppcheck-opensource/cppcheck/tree/a77ef38eca39bd55b2cba982f19812f0920728d2
clang-tidy.yml: use clang to generate the compilation database so we have the proper warning flagshttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/836c8f4c70140d45e9f62f9f38793420461ed60f
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
836c8f4https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/836c8f4c70140d45e9f62f9f38793420461ed60f
https://github.com/cppcheck-opensource/cppcheck/tree/836c8f4c70140d45e9f62f9f38793420461ed60f
oss-fuzz/main.cpp: fixed `-Wmissing-prototypes` warninghttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/2be910728fe1a97ea37bcba7969982ceaee2ab94
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
2be9107https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/2be910728fe1a97ea37bcba7969982ceaee2ab94
https://github.com/cppcheck-opensource/cppcheck/tree/2be910728fe1a97ea37bcba7969982ceaee2ab94
astutils.h: fixed `-Wdocumentation` warninghttps://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/3de0310fd200b84f22801ef48da5e49d60d47a34
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
3de0310https://github.com/cppcheck-opensource/cppcheck/pull/4036/commits/3de0310fd200b84f22801ef48da5e49d60d47a34
https://github.com/cppcheck-opensource/cppcheck/tree/3de0310fd200b84f22801ef48da5e49d60d47a34
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width

URLs of crawlers that visited me.