Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

removed deprecated --showtime= value top5 #5904

Merged
merged 1 commit into from
Jan 22, 2024

Conversation

firewave
Copy link
Collaborator

No description provided.

@firewave firewave changed the title removed deprecated --showtime= value top5 [skip ci] removed deprecated --showtime= value top5 Jan 21, 2024
@firewave firewave marked this pull request as ready for review January 22, 2024 09:32
@chrchr-github chrchr-github merged commit fd9af82 into danmar:main Jan 22, 2024
64 checks passed
@firewave firewave deleted the top5-remove branch January 22, 2024 11:07
@chrchr-github
Copy link
Collaborator

chrchr-github commented Jan 22, 2024

Seems like daca also needs adjustments:

2024-01-22 10:54
ftp://ftp.de.debian.org/debian/pool/main/t/tseries/tseries_0.10-55.orig.tar.gz
cppcheck-options: --library=posix --library=gnu  --showtime=top5 --check-library --inconclusive --enable=style,information --inline-suppr --disable=missingInclude --suppress=unmatchedSuppression --template=daca2 --debug-warnings --suppress=autoNoType --suppress=valueFlowBailout --suppress=bailoutUninitVar --suppress=symbolDatabaseWarning --suppress=valueFlowBailoutIncompleteVar -D__GNUC__ --platform=unix64 -j2
platform: Linux-6.1.0-0.deb11.6-amd64-x86_64-with-glibc2.31
python: 3.9.2
client-version: 1.3.45
compiler: g++ (Debian 10.2.1-6) 10.2.1 20210110
cppcheck: head 2.13.0
head-info: fd9af82 (2024-01-22 11:54:15 +0100)
count: Crash! 25
elapsed-time: -101.0 0.1
head-timing-info:

old-timing-info:
 Tokenizer::createTokens: 0.000557s (avg. 0.000557s - 1 result(s))
 Tokenizer::simplifyTokens1::ValueFlow: 0.000368s (avg. 0.000368s - 1 result(s))
 Tokenizer::simplifyTokens1::createSymbolDatabase: 0.000143s (avg. 0.000143s - 1 result(s))
 Other::runChecks: 0.000125s (avg. 0.000125s - 1 result(s))
 Tokenizer::simplifyTokens1::simplifyTokenList1: 8.5e-05s (avg. 8.5e-05s - 1 result(s))
 Overall time: 0.001609s

head results:
cppcheck: error: unrecognized --showtime mode: 'top5'. Supported modes: file, file-total, summary, top5_file, top5_summary.

DONE

Or is it just clients that are out of date?

@firewave
Copy link
Collaborator Author

firewave commented Jan 22, 2024

Or is it just clients that are out of date?

That.

client-version: 1.3.45

current: CLIENT_VERSION = "1.3.54"

I really need to finish up #5420.

@firewave
Copy link
Collaborator Author

client-version: 1.3.45

That client version is extremely old. 1.3.46 was introduced in July with 682bdd3.

firewave added a commit to firewave/cppcheck that referenced this pull request Jan 22, 2024
chrchr-github pushed a commit that referenced this pull request Jan 22, 2024
This reverts commit fd9af82.

We have a very outdated daca client which is wrecking havoc because it
still relies on this.
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