This line took an hour to crank out. But figured it out thanks to IRC! Can now destructure Option types with comprehensive error messages in a single line. Makes it much nicer to work with Some / None values. https://github.com/datrs/random-access-disk/blob/master/src/sync.rs#L42 …
-
But why return a Result and still have a potential panic in there?
If we ever hit this state we should definitely be panicking - it means that the open() method wasn't called / or our fd was cleaned up when it shouldn't have been. Fatal either way, and very well worth a panic.