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.

Sign up for GitHub

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

Jump to bottom

Error on loading automaton from missing embedded resource #13

Closed
woutersl opened this issue Oct 9, 2014 · 2 comments
Closed

Error on loading automaton from missing embedded resource #13

woutersl opened this issue Oct 9, 2014 · 2 comments
Labels
1.3.0 enhancement New feature or request major Runtime - .Net
Milestone
1.4.0.release

Comments

@woutersl
Copy link
Member

woutersl commented Oct 9, 2014

Original report by Laurent Wouters (Bitbucket: 557058:675792b6-d731-4823-9f7d-c6dfcb2df2b5, ).

When loading an automaton from an embedded resource that is missing, the error experienced by the user is a null reference exception ; which is not very helpful.

An exception with a meaningful error message hinting at how to fix the issue should be thrown instead.
@woutersl
Copy link
Member Author

woutersl commented Oct 9, 2014

Original comment by Laurent Wouters (Bitbucket: 557058:675792b6-d731-4823-9f7d-c6dfcb2df2b5, ).

[fix] Fixed issue #13 by throwing an explicit exception on missing resources

1 similar comment
@woutersl
Copy link
Member Author

woutersl commented Oct 9, 2014

Original comment by Laurent Wouters (Bitbucket: 557058:675792b6-d731-4823-9f7d-c6dfcb2df2b5, ).

[fix] Fixed issue #13 by throwing an explicit exception on missing resources

@woutersl woutersl closed this as completed Oct 9, 2014
@woutersl woutersl added this to the 1.4.0.release milestone Aug 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
1.3.0 enhancement New feature or request major Runtime - .Net
Projects
None yet
Milestone
1.4.0.release
Development

No branches or pull requests
1 participant
@woutersl