[arghwhat]

> https://github.com/servo/servo/search?q=unwrap&unscoped_q=un...

522 results in servo. Granted, some (but not all) are in tests.

`.unwrap()` is expected to be in every code base out there, as there is rarely much point in handling errors such as tainted mutex locks.

[frenchman99]

Most of the code you're linking to is unit tests, not application code.

[arghwhat]

No, I am thinking of application code. If you're trying to handle a poisoned mutex, you're doing something wrong.

[crematoria]

Yeah, most of that code is stuff like examples and tests, and most errors should be forwarded, not unwrapped. But yeah, of course if a resource that you depend on is potentially in an invalid state thanks to a thread crashing, it makes perfect sense to unwrap / expect.