I shot myself in a foot a bit with following code. Would rule for that make sense? `error(exception)...

# detekt

[Peter]

I shot myself in a foot a bit with following code. Would rule for that make sense? error(exception)

[marschwar]

Could you elaborate a bit more what you mean?

[Peter]

I guess proper way should be throw IllegalStateException(exception), so you properly set stacktrace cause?

[Peter]

error(cause) is the same as writing throw IllegalStateException(exception.toString())

[marschwar]

That does sound like a good rule.

[Brais Gabin]

Oh, because error receives Any and you passed the exception and instead of using it as a cause it just called the toString. Right?

[Peter]

yeah, that's how it's implemented

[Brais Gabin]

Yeap! That looks like a good rule for detekt.

[Peter]

Ok, will open GitHub issue after work