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

Add enumsc and enumscstr. #466

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

Add enumsc and enumscstr. #466

wants to merge 1 commit into from

Conversation

phidebian
Copy link

Add enumsc and enumscstr, an enum for start cond instead of define and the optional enum to string array for easier debug printout.

…d the optional enum to string array for easier debug printout
@phidebian
Copy link
Author

I am writting a lex scanner (actually many) with a good number of SC, beeing able to printout the SC in text is a good help in the debug print out. the cost of change is modest.

Note I did this quick'n'dirty, I didn't check if this exit already in other clones.....

Cheers.

@westes
Copy link
Owner

westes commented Jun 5, 2021

Can you see about the conflicts that have ben introduced from the merging in of #489?

@Explorer09
Copy link
Contributor

Flex allows bool type for the enumsc and enumscstr flags. Please use that instead of int. It helps readability.

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

3 participants