Skip to content

Conversation

wolfmanx
Copy link

@wolfmanx wolfmanx commented Aug 7, 2025

This resolves issue #6 by collecting arguments for quoted empty strings.

@TrySound
Copy link
Owner

TrySound commented Aug 8, 2025

@wolfmanx wolfmanx force-pushed the quoted-empty-strings-are-valid-arguments branch from 51f1004 to 4dad341 Compare August 8, 2025 23:58
@wolfmanx
Copy link
Author

wolfmanx commented Aug 9, 2025

I wrote the test.

As you can see, it should not have been written before the unescaping was fixed though (#5).
Once it is fixed, the last two test cases will become invalid, since the double quoting case will result in an empty argument and the single quoting case will have the currently dropped backslash added.

@wolfmanx wolfmanx force-pushed the quoted-empty-strings-are-valid-arguments branch from 4dad341 to 0dffca2 Compare August 9, 2025 13:13
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