[sayrer]

Yes, can't have .unwrap() in production code (it's ok in tests)

[orphea]

Like goto, unwrap is just a tool that has its use cases. No need to make a boogeyman out of it.

[gishh]

To be fair, if you’re not “this tall” you really shouldn’t consider using goto in a c program. Most people aren’t that tall.

[tucnak]

Nonsense. Linux kernel for one example, uses goto everywhere for error handling.

[gishh]

How many codebases besides the Linux kernel had you seen an intentional amount of use of goto?

[orphea]

The answer is one search away on GitHub.

I will give you one example though: various .NET repositories (runtime, aspnetcore, orleans).

[metaltyphoon]

Yes it's meant to be used in test code. If you're sure it can't fail do then use .expect() that way it shows you made a choice and it wasn't just a dev oversight.

[fwjafwasd]

panicans should be using .expect() in production

[keyle]

unwrap itself isn't the problem...