andrew/libakstdlib
1
0
Fork 0
Code Issues 27 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Remove unused snprintf count parameter #42

Open
logikoma wants to merge 1 commits from 10 into main
pull from: 10
merge into: andrew:main
Conversation 0 Commits 1 Files Changed 6 +38 -71
logikoma commented 2026-08-03 16:34:34 -04:00
Collaborator
Copy Link

Closes #32

Remove the unused count parameter from aksl_snprintf and aksl_vsnprintf, update the format tests and public documentation, and retain truncation as AKERR_OUTOFBOUNDS.

Validation: cmake --build build -j2; ctest --test-dir build --output-on-failure.

Closes #32 Remove the unused count parameter from aksl_snprintf and aksl_vsnprintf, update the format tests and public documentation, and retain truncation as AKERR_OUTOFBOUNDS. Validation: cmake --build build -j2; ctest --test-dir build --output-on-failure.
logikoma added 1 commit 2026-08-03 16:34:34 -04:00
Remove unused snprintf count parameter
Some checks failed
libakstdlib CI Build / cmake_build (push) Successful in 2m55s
Details
libakstdlib CI Build / sanitizers (push) Successful in 2m56s
Details
libakstdlib CI Build / mutation_test (push) Failing after 1s
Details
libakstdlib CI Build / coverage (push) Successful in 2m50s
Details
0620370dd9
Some checks failed
libakstdlib CI Build / cmake_build (push) Successful in 2m55s
Required
Details
libakstdlib CI Build / sanitizers (push) Successful in 2m56s
Required
Details
libakstdlib CI Build / mutation_test (push) Failing after 1s
Required
Details
libakstdlib CI Build / coverage (push) Successful in 2m50s
Required
Details
This pull request doesn't have enough required approvals yet. 0 of 1 approvals granted from users or teams on the allowlist.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin 10:10
git checkout 10
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
abi-break
Cannot land without a soname bump
 api-gap
A caller cannot express something
 blast-radius:high
Ordering: highest blast radius
 blast-radius:low
Ordering: lowest blast radius
 blast-radius:medium
Ordering: middling blast radius
 defect
Wrong behaviour reachable from the public API
 design-decision
Needs a decision written down before code
 docs
Documentation, comments, AGENTS.md
 hygiene
Style, dead code, consistency
 packaging
Build, install, pkg-config, CMake config
 performance
Has a measurement behind it
 status::blocked
Waiting on something else -- a decision, a dependency, or another issue
 status::grooming
Not yet worked out: scope, approach or priority still being settled
 status::ready
Understood well enough to start: scope and approach are settled
 test-coverage
A gap in the suites, not a defect
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
logikoma tachikoma andrew
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: andrew/libakstdlib#42
Delete Branch "10"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?