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 methods to OhmNode and add case labels #57

Open
wants to merge 6 commits into
base: new-whitespace-handling
Choose a base branch
from

Conversation

felixauringer
Copy link

  • We added methods to check whether an OhmNode has no content. If that is the case, it is safe to ignore the node and its children when building the source map. This prevents all index out of bounds errors in the source map that we have found so far (all our comment-tests are green with this branch).
  • We added a method firstChild to OhmNode because we need this operation very often.
  • We added some case labels in order to write SemanticActions for these cases.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.03%) to 85.271% when pulling 774c995 on felixauringer:swt-submission into d7f2ac8 on hpi-swa:new-whitespace-handling.

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