Skip to content
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.

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

fix: implement varargs for ESC commands. 'custom' makes use of this #592 #592

Merged
merged 4 commits into from
May 2, 2022

Conversation

BHSDuncan
Copy link
Collaborator

@BHSDuncan BHSDuncan commented May 2, 2022

@BHSDuncan BHSDuncan changed the title First pass at implementing varargs for ESC commands. makes use of this. WIP: first pass at implementing varargs for ESC commands. 'custom' makes use of this. May 2, 2022
@StraToN
Copy link
Collaborator

StraToN commented May 2, 2022

Exactly what I wanted ;)
Thanks!

BHSDuncan added 3 commits May 2, 2022 11:41
…ed immediately by a space, or a space followed immediately by a quotation mark
@BHSDuncan BHSDuncan requested review from StraToN and balloonpopper May 2, 2022 18:04
@BHSDuncan BHSDuncan marked this pull request as ready for review May 2, 2022 18:05
@BHSDuncan BHSDuncan requested a review from dploeger as a code owner May 2, 2022 18:05
@BHSDuncan BHSDuncan changed the title WIP: first pass at implementing varargs for ESC commands. 'custom' makes use of this. fix: implementing varargs for ESC commands. 'custom' makes use of this. May 2, 2022
@BHSDuncan
Copy link
Collaborator Author

Be advised that this also fixes an issue with spaces immediately after (or immediately before) a quotation mark. This affects all ESC commands that can take arguments.

@StraToN StraToN changed the title fix: implementing varargs for ESC commands. 'custom' makes use of this. fix: implement varargs for ESC commands. 'custom' makes use of this #592 May 2, 2022
@StraToN StraToN merged commit 738ce0d into develop May 2, 2022
@StraToN StraToN deleted the issue-106 branch May 2, 2022 20:50
@StraToN
Copy link
Collaborator

StraToN commented May 2, 2022

Thanks!

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.

custom command crashes if the function takes no parameters
3 participants