Skip to content

Commit

Permalink
Fix doskey macros for inputs with >1 consecutive whitespace (#17129)
Browse files Browse the repository at this point in the history
Initially the PR restored the original v1 conhost code for
`MatchAndCopyAlias` which fixed the linked issue.
Afterwards, I've taken the liberty to rewrite the code to use modern
constructs again, primarily `string_view`. Additionally, the v1 code
first counted the number of arguments and then iterated through it
again to assemble them. This new code does both things at once.

Closes #15736

## Validation Steps Performed
The unit tests have been extended to cover multiple consecutive
spaces. All tests pass.
  • Loading branch information
lhecker authored May 2, 2024
1 parent 8dd4512 commit c52dca4
Show file tree
Hide file tree
Showing 5 changed files with 118 additions and 613 deletions.
Loading

0 comments on commit c52dca4

Please sign in to comment.