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.

Sign up for GitHub

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

Jump to bottom

added C preprocessor #721

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

added C preprocessor #721

wants to merge 1 commit into from

Conversation

theishshah
Copy link

supports #include #define #ifdef #endif

will not highlight the contents following #{statement}

highlights in Comment:Preproc

@theishshah
added preprocessor
6c83572 
supports #include #define #ifdef #endif
@gfx gfx changed the title added preprocessor added C preprocessor Sep 13, 2017
vidarh
vidarh suggested changes Jan 17, 2019
View reviewed changes
Copy link
Contributor

@vidarh vidarh left a comment

Choose a reason for hiding this comment

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

Hi, suggest you close this. More complete preprocessor directive support is added in the :macros state in the current C-lexer. Thanks.

@pyrmont
Copy link
Contributor

pyrmont commented Jan 12, 2020

@theishshah Hi, I'm sorry it's taken so long to get to this. Given it was more than 2 years ago, I'm not sure if this is still something you remember/need, but do you remember why you were trying to add it? As @vidarh mentioned in his comment from last year, the C lexer has support for preprocessor directives.

@pyrmont pyrmont added the author-action The PR has been reviewed but action by the author is needed label Jan 12, 2020
@pyrmont pyrmont self-assigned this Jan 12, 2020
@pyrmont pyrmont removed their assignment Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vidarh vidarh vidarh requested changes

Assignees
No one assigned
Labels
author-action The PR has been reviewed but action by the author is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@theishshah @pyrmont @vidarh