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

Update the Tera Term macro lexer (new 'setspeed' command) #1337

Merged
merged 1 commit into from
May 22, 2020

Conversation

kurtmckee
Copy link
Contributor

I've added the setspeed command to the Tera Term macro lexer.

In addition, I removed the analyse_text() math and just return 0.01 or 0.0 directly. I've also expanded and alphabetized the regex strings so that it's easier to update the macro commands and variables if any additional commands are added in the future.

Please let me know if I've overlooked anything or need to make changes to this update.

@kurtmckee
Copy link
Contributor Author

Haha, neat! I got PR number '1337'. 😃

@Anteru
Copy link
Collaborator

Anteru commented Dec 23, 2019

The default for analyse_text is to return 0 (well, None, and that gets converted to 0 ...). Unless there is a warning that it's missing due to ambiguity, I'd remove it.

@Anteru Anteru self-assigned this May 22, 2020
@Anteru Anteru added this to the 2.7 milestone May 22, 2020
@Anteru Anteru merged commit f17ca94 into pygments:master May 22, 2020
@kurtmckee kurtmckee deleted the update-teraterm-lexer branch March 3, 2021 14:19
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.

None yet

2 participants