René's URL Explorer Experiment


Title: reduced usage of mutable `Settings` objects in tests by firewave · Pull Request #4798 · cppcheck-opensource/cppcheck · GitHub

Open Graph Title: reduced usage of mutable `Settings` objects in tests by firewave · Pull Request #4798 · cppcheck-opensource/cppcheck

X Title: reduced usage of mutable `Settings` objects in tests by firewave · Pull Request #4798 · 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: Currently the Settings objects in the tests are all mutable. This might cause some tests to modify the settings as they also contain run-time information which will be used by subsequent tests usin...

X Description: Currently the Settings objects in the tests are all mutable. This might cause some tests to modify the settings as they also contain run-time information which will be used by subsequent tests usin...

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

X: @github

direct link

Domain: github.com

route-pattern/:user_id/:repository/pull/:id/commits(.:format)
route-controllerpull_requests
route-actioncommits
fetch-noncev2:267520c4-325d-a623-08ad-b8dfbb01d54c
current-catalog-service-hashae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b
request-idB106:D3583:F2482B:151B061:6A4DF108
html-safe-nonce838f1fef9c1805561a9fb7eb7d61cbc74da7da9e785b5453755442fc9103f74d
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCMTA2OkQzNTgzOkYyNDgyQjoxNTFCMDYxOjZBNERGMTA4IiwidmlzaXRvcl9pZCI6IjU1NTI5MzkwOTgyNDU1NTg1MzYiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ==
visitor-hmac5775bf03eea652361ded718cf817ac4f38ff8b54d932b090bb631b63b7247448
hovercard-subject-tagpull_request:1243634668
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/4798/commits
twitter:imagehttps://opengraph.githubassets.com/c327649bf1549e68dbed649d42d78e9dec90119498bb750a90f419291618bf3e/cppcheck-opensource/cppcheck/pull/4798
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/c327649bf1549e68dbed649d42d78e9dec90119498bb750a90f419291618bf3e/cppcheck-opensource/cppcheck/pull/4798
og:image:altCurrently the Settings objects in the tests are all mutable. This might cause some tests to modify the settings as they also contain run-time information which will be used by subsequent tests usin...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernamefirewave
hostnamegithub.com
expected-hostnamegithub.com
None5818716c93c6a2925b815402541a32814e43a7b1261c322b0c2df75224289566
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
releasef4bb89367ca678f057d79b1abc45d6675b1bd5b2
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fcppcheck-opensource%2Fcppcheck%2Fpull%2F4798%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%2F4798%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/4798/commits
Reloadhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits
Reloadhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits
Please reload this pagehttps://github.com/cppcheck-opensource/cppcheck/pull/4798/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:settingshttps://github.com/firewave/cppcheck/tree/settings
Conversationhttps://github.com/cppcheck-opensource/cppcheck/pull/4798
Commits18 (18)https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits
Checkshttps://github.com/cppcheck-opensource/cppcheck/pull/4798/checks
Files changedhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/files
reduced usage of mutable Settings objects in testshttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits#top
danmarhttps://github.com/danmar
cppcheck-opensource:mainhttps://github.com/cppcheck-opensource/cppcheck/tree/main
firewave:settingshttps://github.com/firewave/cppcheck/tree/settings
TestFixture: immediately bail out when library could not be loadedhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/36b62ad9f6ae6aa46f0dbc4766df1dea65d24702
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
36b62adhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/36b62ad9f6ae6aa46f0dbc4766df1dea65d24702
https://github.com/cppcheck-opensource/cppcheck/tree/36b62ad9f6ae6aa46f0dbc4766df1dea65d24702
TestFixture: added `SettingsBuilder` to allow inline initialization of objectshttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/944cce52204ef4a6c9537fcc3397c687e1214640
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
944cce5https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/944cce52204ef4a6c9537fcc3397c687e1214640
https://github.com/cppcheck-opensource/cppcheck/tree/944cce52204ef4a6c9537fcc3397c687e1214640
made some `Settings` objects in tests `const`https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/de94a7d4a7b5779e8c21f37990c71409074f3a7a
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
de94a7dhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/de94a7d4a7b5779e8c21f37990c71409074f3a7a
https://github.com/cppcheck-opensource/cppcheck/tree/de94a7d4a7b5779e8c21f37990c71409074f3a7a
first round of `SettingsBuilder` usage in testshttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/8b88e704fc4545c85e4672f5265f286578812791
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
8b88e70https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/8b88e704fc4545c85e4672f5265f286578812791
https://github.com/cppcheck-opensource/cppcheck/tree/8b88e704fc4545c85e4672f5265f286578812791
moved `givenACodeSampleToTokenize::settings` instance to proper filehttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/5783bed36dba2322f31831c44bfad332bfb9e658
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
5783bedhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/5783bed36dba2322f31831c44bfad332bfb9e658
https://github.com/cppcheck-opensource/cppcheck/tree/5783bed36dba2322f31831c44bfad332bfb9e658
made some local `const Settings` objects in test `static`https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/b8079ac48fb3bb7ff8eb498cef2e2b54d2c2ddbb
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
b8079achttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/b8079ac48fb3bb7ff8eb498cef2e2b54d2c2ddbb
https://github.com/cppcheck-opensource/cppcheck/tree/b8079ac48fb3bb7ff8eb498cef2e2b54d2c2ddbb
TestBufferOverrun: added TODO about missing settings resethttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/8109781c7c7211696c134d7a4b4f2509b0e85e56
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
8109781https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/8109781c7c7211696c134d7a4b4f2509b0e85e56
https://github.com/cppcheck-opensource/cppcheck/tree/8109781c7c7211696c134d7a4b4f2509b0e85e56
added TODO about `exename` not yet being set in `TestFixture::SettingsBuilder::library()`https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/d46fcc8475d352deb266a8b69a5f0b5bce551135
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
d46fcc8https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/d46fcc8475d352deb266a8b69a5f0b5bce551135
https://github.com/cppcheck-opensource/cppcheck/tree/d46fcc8475d352deb266a8b69a5f0b5bce551135
small test classes initialization cleanuphttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/fedf5690101bb41d9d70772d9510c486c216d146
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
fedf569https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/fedf5690101bb41d9d70772d9510c486c216d146
https://github.com/cppcheck-opensource/cppcheck/tree/fedf5690101bb41d9d70772d9510c486c216d146
TestIO: fixed performance regressionhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/67009a242dda0beee81725de17f1397fd384608d
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
67009a2https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/67009a242dda0beee81725de17f1397fd384608d
https://github.com/cppcheck-opensource/cppcheck/tree/67009a242dda0beee81725de17f1397fd384608d
testclass.cpp: adjustments after `Preprocessor` was added by another PRhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/efa8945e0cf502795bf24d52cb91da6ffc0ea1b3
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
efa8945https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/efa8945e0cf502795bf24d52cb91da6ffc0ea1b3
https://github.com/cppcheck-opensource/cppcheck/tree/efa8945e0cf502795bf24d52cb91da6ffc0ea1b3
testtokenlist.cpp: fixed `functionConst` selfcheck warninghttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/61338a68c93e1d56ffd39cd775bb6c9f202f2927
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
61338a6https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/61338a68c93e1d56ffd39cd775bb6c9f202f2927
https://github.com/cppcheck-opensource/cppcheck/tree/61338a68c93e1d56ffd39cd775bb6c9f202f2927
teststl.cpp: removed `const` to fix compilation after changes in another mergehttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/7f505165fb7ab6a9617a413e5f5fd13243fab59f
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
7f50516https://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/7f505165fb7ab6a9617a413e5f5fd13243fab59f
https://github.com/cppcheck-opensource/cppcheck/tree/7f505165fb7ab6a9617a413e5f5fd13243fab59f
testtokenlist.cpp: fixed `functionConst` selfcheck warningshttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/a0a0acf472ab1561379b4bc0427bd20e3adf0d0f
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
a0a0acfhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/a0a0acf472ab1561379b4bc0427bd20e3adf0d0f
https://github.com/cppcheck-opensource/cppcheck/tree/a0a0acf472ab1561379b4bc0427bd20e3adf0d0f
do not use `static const` settings in testshttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/935755f0134d2e658aa37f16052ec24278d4fd0c
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
935755fhttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/935755f0134d2e658aa37f16052ec24278d4fd0c
https://github.com/cppcheck-opensource/cppcheck/tree/935755f0134d2e658aa37f16052ec24278d4fd0c
testthreadexecutor.cpp: removed `const` to fix compilation after changes in another mergehttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/cea46ae042259261dfb21457824c79afd092fd9b
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
cea46aehttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/cea46ae042259261dfb21457824c79afd092fd9b
https://github.com/cppcheck-opensource/cppcheck/tree/cea46ae042259261dfb21457824c79afd092fd9b
dmakehttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/dde6eea3a4885c72cbc226295d4b8880f7468875
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
dde6eeahttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/dde6eea3a4885c72cbc226295d4b8880f7468875
https://github.com/cppcheck-opensource/cppcheck/tree/dde6eea3a4885c72cbc226295d4b8880f7468875
fixed `helpers.cpp` compilation without precompiled headershttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/aaecc8cebfba809002ff5113686314c014449e20
https://github.com/firewave
firewavehttps://github.com/cppcheck-opensource/cppcheck/commits?author=firewave
aaecc8chttps://github.com/cppcheck-opensource/cppcheck/pull/4798/commits/aaecc8cebfba809002ff5113686314c014449e20
https://github.com/cppcheck-opensource/cppcheck/tree/aaecc8cebfba809002ff5113686314c014449e20
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.