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

GHC Cmm #1387

Merged
merged 102 commits into from Apr 14, 2020
Merged

GHC Cmm #1387

merged 102 commits into from Apr 14, 2020

Conversation

supersven
Copy link
Contributor

This is a lexer for C minus minus, which is a dialect of C--. It's used as intermediate and hand-written language in the Glasgow Haskell Compiler.

More details:

@pyrmont pyrmont self-assigned this Jan 12, 2020
@pyrmont pyrmont added the needs-review The PR needs to be reviewed label Jan 12, 2020
Copy link
Contributor

@pyrmont pyrmont left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry it's taken so long to get to this :( I hope you haven't lost interest!

It's very cleanly defined but it does feel a little like a proper language parser rather than a syntax highlighter. Would it not be possible to have a simpler number of rules that match keywords individually rather than looking to match sets of keywords with arbitrary whitespace and comments between them?

lib/rouge/lexers/ghc_cmm.rb Outdated Show resolved Hide resolved
lib/rouge/lexers/ghc_cmm.rb Outdated Show resolved Hide resolved
lib/rouge/lexers/ghc_cmm.rb Show resolved Hide resolved
lib/rouge/lexers/ghc_cmm.rb Outdated Show resolved Hide resolved
@pyrmont pyrmont added author-action The PR has been reviewed but action by the author is needed and removed needs-review The PR needs to be reviewed labels Apr 5, 2020
@supersven
Copy link
Contributor Author

Hey @pyrmont ,
No worries, I saw that you were very busy with many PRs...

Thanks a lot for taking the time for this review!

I will check, if I can simplify the keyword matching rules. (I first have to remember what I actually did ... 😉 )

@pyrmont
Copy link
Contributor

pyrmont commented Apr 8, 2020

@supersven This looks good to me. Did you have anything else you wanted to add?

@supersven
Copy link
Contributor Author

@pyrmont I'm doing some final testing ...

I've pushed a small change regarding CLOSURE.

@pyrmont
Copy link
Contributor

pyrmont commented Apr 9, 2020

@supersven No worries, just say when! :)

@supersven
Copy link
Contributor Author

@pyrmont ,

I've finished testing: I think it can now be merged. 😃

Thanks a lot for your time and this review!

(Please squash the commits during merge.)

@supersven supersven closed this Apr 11, 2020
@supersven supersven reopened this Apr 11, 2020
Copy link
Contributor

@pyrmont pyrmont left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for not raising this before (especially after I said this was good to go) but I just noticed this:

lib/rouge/lexers/ghc_cmm.rb Outdated Show resolved Hide resolved
@pyrmont pyrmont merged commit 4b001ac into rouge-ruby:master Apr 14, 2020
@pyrmont pyrmont removed the author-action The PR has been reviewed but action by the author is needed label Apr 14, 2020
@pyrmont
Copy link
Contributor

pyrmont commented Apr 14, 2020

@supersven Thanks for all the work you put into this. The lexer will be part of the next version of Rouge, v3.18.0. That version is scheduled for release on Tuesday 14 April (so either today or tomorrow depending on where you're reading this!).

mattt pushed a commit to NSHipster/rouge that referenced this pull request May 21, 2020
This commit adds a lexer for GHC Cmm.
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