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

Completely review exception handling #16

Open
jp2masa opened this issue Oct 21, 2017 · 3 comments
Open

Completely review exception handling #16

jp2masa opened this issue Oct 21, 2017 · 3 comments

Comments

@jp2masa
Copy link
Member

jp2masa commented Oct 21, 2017

From @charsleysa on January 4, 2016 8:39

Exception filtering is not yet supported.

For an example of Exception filtering in C# see http://pastebin.com/URkYjsiE

Copied from original issue: CosmosOS/Cosmos#320

@jp2masa
Copy link
Member Author

jp2masa commented Oct 21, 2017

From @mterwoord on January 4, 2016 8:41

Lets also include in this issue:

  • Finally clause
  • generic testing

@jp2masa
Copy link
Member Author

jp2masa commented Oct 9, 2018

Note from #64 (see also #64 (comment)):
When throwing an exception, we should empty the stack (if there's no item in the stack, do nothing). I think it would be a good idea to add some common infrastructure to throw exceptions.

@quajak
Copy link
Member

quajak commented Nov 8, 2018

It would also be helpful if exceptions in using blocks are also supported. As they right now lead to crashes (CPU Exception 6).
Example code: https://github.com/CosmosOS/Cosmos/blob/8b7613e2d81b004e4af72210a7afa60895818231/Tests/Kernels/Cosmos.Compiler.Tests.Exceptions/Kernel.cs#L167

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants