search

salmanahmad / silo

The Silo Programming Language
0 stars 0 forks source link

Exceptions do not Propagate with CLI #40

Closed salmanahmad closed 10 years ago

salmanahmad commented 10 years ago

I had a file that looked like:

options : silo.lang.FooBarBaz = silo.lang.FooBarBaz()

When run with silo file.silo no exception was reported that "FooBarBaz" does not exist.

salmanahmad commented 10 years ago

I should also inspect Runtime.spawn and the following lines:

...
        } catch(Exception e) {
            // TODO: Better error handling.
            // TODO: This is potentially bad.
            e.printStackTrace();
            throw new RuntimeException(e);
        }
...
salmanahmad commented 10 years ago

Fixed in 525ea3a8d7d211a616709234b3b353f9841588f7. If you want to export the exception to Java-land and to threads waiting on "Actor#await" feel free to open a new issue.