やったこと

#4779

unreachable!の実装。わりと前に作ってあったのだけどPull Requestにまとめるのがめんどうで出していなかった。

で、出したら出したですぐに@RX14が「I think it would be best to start using unreachable! in this PR, just in a seperate commit.」って言ってくる。個人的にはどうしても分けて考えた方がいいような気がしていたのだけど、上手く言語化できないのでとりあえずraise "BUG: をunreachable!で置き換えてみることにする。

すると今度は@straight-shootaから色々言われる。それで思ったのがCrystalの開発陣は当然Crystalに精通してるので、unreachable!が確実に必要になる場面を慎重に回避する傾向があるんじゃないか、ということ。その辺を踏まえて、今のところraise "BUG: を置き換えるあたりはrevertしている。