Skip to content

libtests: update format strings to avoid casts, drop some macros #18106

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.

Sign up for GitHub

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

Jump to bottom
Closed
vszakats wants to merge 27 commits into from
Closed

libtests: update format strings to avoid casts, drop some macros #18106

vszakats wants to merge 27 commits into from

Conversation

@vszakats
Copy link
Member

@vszakats vszakats commented Jul 30, 2025 •
edited
Loading

  • bump format strings to show the full value, drop casts.
  • drop redundant casts (enum → %d).
  • drop some single-use macros.
  • replace int with bool in testtrace.

@vszakats vszakats marked this pull request as draft July 30, 2025 16:25
@github-actions github-actions bot added the tests label Jul 30, 2025
@vszakats vszakats marked this pull request as ready for review July 30, 2025 17:48
vszakats added 25 commits July 31, 2025 00:45
@vszakats
lib557: drop single use macros
00e6e7d
@vszakats
lib540: replace macros with local vars
9b4aed3
@vszakats
lib540: lowercase
22b1a4e
@vszakats
cli_h2_pausing.c printf format
fb51f82
@vszakats
cli_h2_serverpush.c printf format
2c35497
@vszakats
cli_hx_download.c printf format
a60fb6b
@vszakats
cli_ws_data.c printf format
b53c59f
@vszakats
cli_ws_pingpong.c printf format
1d21296
@vszakats
cli_hx_upload.c fix indent
f4cfaff
@vszakats
lib???? printf masks 1
4b82424
@vszakats
lib1520.c drop single-use macros, #if 0
e22b40b
@vszakats
lib506.c convert macro to local variable
6e0fda0
@vszakats
lib1507.c drop single-use macros
41ecb52
@vszakats
cli_ws_data.c use CURL_FORMAT_CURL_OFF_T
7eee35d
@vszakats
lib??? printf masks 1
7ee583f
@vszakats
libtests finalize 1
bc51b41
@vszakats
libtests finalize 1
75cb2cf
@vszakats
libtests finalize 1
de6a940
@vszakats
unit1309.c formatting
3c239a9
@vszakats
checksrc
93388ba
@vszakats
some more
1a0665f
@vszakats
cleanup
1c1fb34
@vszakats
libtests: use bool in debug_config
34dbb2a
@vszakats
lib536 replace macros with local vars
f8851b4
@vszakats
lib547.c replace macro with ctatic const, strlen with sizeof
4ec2132
@vszakats vszakats closed this in 37913c0 Jul 31, 2025
@vszakats vszakats deleted the t-less-macros branch July 31, 2025 07:35
@vszakats vszakats mentioned this pull request Aug 2, 2025
Closed
vszakats added a commit that referenced this pull request Aug 4, 2025
@vszakats
libtests: use FMT_SOCKET_T, drop more casts
c2fed8c 
Follow-up to 37913c0 #18106

Closes #18142
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

tests

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vszakats