eclipse-sirius / sirius-emf-json

JSON-based EMF Resource implementation - part of Eclipse Sirius
https://eclipse.dev/sirius/sirius-web.html
Eclipse Public License 2.0
5 stars 10 forks source link

[36] Avoid using costly Exception for package not found errors #40

Closed pcdavid closed 1 month ago

pcdavid commented 1 month ago

Bug: https://github.com/eclipse-sirius/sirius-emf-json/issues/36 Signed-off-by: Pierre-Charles David pierre-charles.david@obeo.fr

pcdavid commented 1 month ago

I've only converted PackageNotFoundException for the moment. If we think this is the right approach, it should probably be generalized to the other JsonException subtypes.

Otherwise, if we decide we need these to be exceptions, another approach will be needed. Maybe add an optional flag to disable this reporting?