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

macro "times" command to repeat macros doesn't work #526

Closed
lexxish opened this issue May 18, 2019 · 6 comments
Closed

macro "times" command to repeat macros doesn't work #526

lexxish opened this issue May 18, 2019 · 6 comments
Assignees
Labels
Bug Unexpected behavior from existing features.

Comments

@lexxish
Copy link
Contributor

lexxish commented May 18, 2019

The command is logged properly as Recorded Macro: macro [times <n>] but macro only runs once.

@LexiconCode LexiconCode added the Bug Unexpected behavior from existing features. label May 18, 2019
@LexiconCode
Copy link
Member

I can reproduce this behavior

@lexxish
Copy link
Contributor Author

lexxish commented May 23, 2019

@LexiconCode any idea what's wrong here? I will definitely want to leverage this functionality for other issues I'm working on. I looked at it but didn't see anything obvious

@seekM
Copy link
Contributor

seekM commented May 27, 2019

It doesn't work for me as well. I found the times spec, but not where its functionality is implemented, any idea?

@mrob95
Copy link
Member

mrob95 commented May 27, 2019

Try altering this line:

mapping[ascii_str] = R(

So that it looks like:

R(Playback(), rdescript="")*Repeat(extra="n")

(ie just add a repeat at the end)
I don't have time to test it at the moment but if it works the way I think it does then that should fix it.

@seekM
Copy link
Contributor

seekM commented May 27, 2019

Thank you, this did the trick for me!

@LexiconCode
Copy link
Member

@lexxish fixed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Unexpected behavior from existing features.
Projects
None yet
Development

No branches or pull requests

4 participants