Skip to content

libssh2: tidy up a knownhost function - #19479

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/libssh2-knownhost
Closed

bagder wants to merge 2 commits into
masterfrom
bagder/libssh2-knownhost

Conversation

@bagder

@bagder bagder commented Nov 12, 2025

Copy link
Copy Markdown
Member
  • replace atoi() with curlx_str_number()
  • use array instead of pointers
  • narrow variable scopes

@testclutch

Copy link
Copy Markdown

Analysis of PR #19479 at ad41c762:

Test 2405 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 10 different CI jobs (the link just goes to one of them).

Test 1904 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 12 different CI jobs (the link just goes to one of them).

Test 853 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 12 different CI jobs (the link just goes to one of them).

Generated by Testclutch

@bagder
bagder marked this pull request as ready for review November 12, 2025 09:47
@bagder bagder closed this in 7428060 Nov 12, 2025
bagder added a commit that referenced this pull request Nov 12, 2025
@bagder
bagder deleted the bagder/libssh2-knownhost branch November 12, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants