> Что именно вам кажется костылями в этом коде?На момент написания коммента я этого патча в глаза не видел.
Просто поддерживаю анонима, написавшего, что _иногда_ нормальное исправление может занять длительное время, а то, что делается быстро, бывает костылём, который говорит "тут ошибки нету, можете больше ничего не делать и не пытаться понять, почему ошибка возникла". И что позиция "тут мне выпустили патч сразу, а там прождали несколько месяцев, поэтому тут сидят хорошие, а там - плохие" сама по себе неверна.
Часто, например, когда в функции происходит разыменование нулевого указателя, многие просто вставляют проверку на NULL и возврат ошибки, не разбираясь, как в неё вообще попал NULL.
Что касается конкретно этого патча - то с моими нулевыми познаниями исходников mariadb я ничего плохого не нашёл, но с оговоркой, что это код оптимизатора. А оптимизаторы сами по себе, как правило, являются довольно запутанным набором костылей, рассматривающих частные случаи и пытающиеся в этих случаях избавиться от каких-то ненужных вычислений. Так что костылём больше костылём меньше - никто не заметит.