Skip to content

proxy/handler: tweak arg handling style and format#48268

Merged
jkarneges merged 2 commits intomainfrom
jkarneges/args-nits
Sep 26, 2025
Merged

proxy/handler: tweak arg handling style and format#48268
jkarneges merged 2 commits intomainfrom
jkarneges/args-nits

Conversation

@jkarneges
Copy link
Member

@jkarneges jkarneges commented Sep 23, 2025

This does a few small things:

  • Use tabs instead of spaces in C++ source files.
  • Ensure files end with a newline.
  • Remove some errant whitespace.
  • Drop get prefix from getters.
  • Mark test functions static.
  • Some #include block reordering/simplifying.
  • Update the args structs a little bit, such as adding verbose fields, and changing Option<Vec<String>> to Vec<String> which clap still counts as optional.

@jkarneges jkarneges changed the title proxy/handler: some style and format tweaks for arg handling proxy/handler: tweak arg handling style and format Sep 26, 2025
@jkarneges jkarneges requested a review from a team September 26, 2025 18:52
@jkarneges jkarneges merged commit e86f5ab into main Sep 26, 2025
21 of 22 checks passed
@jkarneges jkarneges deleted the jkarneges/args-nits branch September 26, 2025 19:28
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