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

Highlight shortened lists for syzlang DSL #1808

Merged
merged 1 commit into from
Feb 24, 2022
Merged

Highlight shortened lists for syzlang DSL #1808

merged 1 commit into from
Feb 24, 2022

Conversation

xairy
Copy link
Contributor

@xairy xairy commented Feb 19, 2022

Update the lexer for syzlang DSL to allow processing inputs with lists and syscall arguments replaced with .... This is useful for highlighting snippets that are shortened for readability.

Also add tests for the new behavior.

Update the lexer for syzlang DSL to allow processing inputs with lists and
syscall arguments replaced with "...". This is useful for highlighting snippets
that are shortened for readability.

Also add tests for the new behavior.

Signed-off-by: Andrey Konovalov <andreyknvl@gmail.com>
@xairy
Copy link
Contributor Author

xairy commented Feb 23, 2022

Hi @tancnle! Could you please take a look at this pull request when you get a chance? It's a short improvement for the syzkaller-related lexers you merged recently - should be straightforward to review. Thanks!

@tancnle
Copy link
Collaborator

tancnle commented Feb 24, 2022

Nice one. Thanks @xairy. The change LGTM 🚀

@tancnle tancnle merged commit df4477e into rouge-ruby:master Feb 24, 2022
razetime pushed a commit to razetime/rouge that referenced this pull request Jun 30, 2022
Update the lexer for syzlang DSL to allow processing inputs with lists and
syscall arguments replaced with "...". This is useful for highlighting snippets
that are shortened for readability.

Also add tests for the new behavior.

Signed-off-by: Andrey Konovalov <andreyknvl@gmail.com>
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