Reduce the usage of `unwrap()`

kud1ing / rucaja

Calling the JVM from Rust via JNI

https://docs.rs/rucaja

Apache License 2.0

31 stars 7 forks source link

Open kud1ing opened 6 years ago

kud1ing commented 6 years ago

We use unwrap() in some places where the result should "in theory" always be existant.

We should handle those unlikely cases. Consider:

bubble Nones up in functions that return an Option. Consider using ? for Options in Rust 1.22.
panic if JVM methods fail when we can not handle it otherwise