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

Added support for @code ... @endocde commands #9

Merged
merged 4 commits into from
May 28, 2024

Conversation

nikarh
Copy link
Contributor

@nikarh nikarh commented Mar 9, 2024

Adds support for doxygen code commands

Notable changes:

  • Added \t char as a possible whitespace
  • Whitespaces used to be collapsed in the lexer. Since code blocks should retain all formatting, lexer emits Whitespace items for every whitespace instead. Now they are collapsed in the parser, which checks if the last GrammarItem ends with a whitespace
  • @code can optionally have an additional parameter (lang) passed with brackets as in @code{.py}. For that, I changed the parameter parsing logic to parse parameters separated with whitespace (including multiple whitespace lexemes) and inside of the brackets.

@Techie-Pi
Copy link
Owner

I'll try to review this PR in the weekend, thank you for your patience! :)

@Techie-Pi Techie-Pi self-assigned this Mar 11, 2024
@Techie-Pi Techie-Pi self-requested a review March 11, 2024 19:50
@pheki
Copy link

pheki commented May 11, 2024

Hi! Any updates @Techie-Pi? We've been waiting on this for vita-rust/vitasdk-sys#30

@Techie-Pi Techie-Pi merged commit cdabd6f into Techie-Pi:main May 28, 2024
1 check passed
@Techie-Pi
Copy link
Owner

Techie-Pi commented May 28, 2024

Sorry, I've not been able to review this PR sooner, lots of exams...
I'll try to publish a new release in the following weeks. In the meantime you could depend on the repo itself.

@pheki
Copy link

pheki commented Jun 7, 2024

No worries and TYSM!

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.

3 participants