えっと
assert(other == nullptr);
を
if( other == nullptr ){
assert(false);
return 0.0f;
}
にしたらダメだとコメントされてるのよね？
でもNDEBUGを定義した後のことを考えて
assert(other == nullptr);
if( other == nullptr ){
return 0.0f;
}
をしたいわけでother == nullptrが重複するのが悪いだろ
記事の書き方のほうがいい