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

[454629] Record label attribute validation support for DotEditor #8

Conversation

prggz
Copy link
Contributor

@prggz prggz commented Nov 20, 2017

  • Implemented validation methods for the recordLabel in
    DotJavaValidator.java
  • Introduced ConvertingValidationMessageAcceptor to accept issues in the
    scope of the subgrammar and register them in the scope of the Dot
    grammar.
  • Handling calculating offset in ConvertingValidationMessageAcceptor
  • Implemented further validation test cases:
  • Tests for other semantic issue (no node returned case) and Syntax
    issues added.

Signed-off-by: Zoey Gerrit Prigge zoey.prigge@uni-duesseldorf.de
Bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=454629

- Implemented validation methods for the recordLabel in
DotJavaValidator.java
- Introduced ConvertingValidationMessageAcceptor to accept Issues in the
scope of the subgrammar and register them in the scope of the Dot
grammar.
- Handling calculating offset in ConvertingValidationMessageAcceptor
- Implemented further validation test cases:
- Tests for other semantic issue (no node returned case) and Syntax
issues added.


Signed-off-by: Zoey Gerrit Prigge <zoey.prigge@uni-duesseldorf.de>
Bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=454629
@eclipsewebmaster
Copy link

Issue tracker reference:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=454629

@miklossy miklossy merged commit 6af54da into eclipse:master Nov 23, 2017
@prggz prggz deleted the 454629_RecordLabel_Validation_DotJavaValidator_And_Tests branch March 9, 2018 11:03
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