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

Prevent random exceptions when Emfatic is unable to provide a textual model view for EMF #1175

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

tsaglam
Copy link
Member

@tsaglam tsaglam commented Jul 5, 2023

Emfatic has no proper error handling. Thus it just throws random unchecked exceptions if it does not like the input model.
However, JPlag can recover from this situation via a parsing exception by skipping this submission. Thus this PR implements an ugly workaround to deal with Emfatic.

@tsaglam tsaglam added bug Issue/PR that involves a bug minor Minor issue/feature/contribution/change language PR / Issue deals (partly) with new and/or existing languages for JPlag labels Jul 5, 2023
@sonarcloud
Copy link

sonarcloud bot commented Jul 5, 2023

[JPlag Plagiarism Detector] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

50.0% 50.0% Coverage
0.0% 0.0% Duplication

@tsaglam tsaglam marked this pull request as ready for review July 5, 2023 13:37
@tsaglam tsaglam merged commit 183d16c into develop Jul 6, 2023
@tsaglam tsaglam deleted the emfatic-error-fix branch July 6, 2023 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue/PR that involves a bug language PR / Issue deals (partly) with new and/or existing languages for JPlag minor Minor issue/feature/contribution/change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant