Ширина сенсора вполне себе сигнатура - мне это тоже показалось прикольным, но в общем без негативных эмоций. А вот всякие diff -= (1 << len) - !shl; меня просто в ступор вводят. Т.е. если ты так пишешь, то хотя бы комментарии ставь, а если не любишь комментировать, то пиши простыми развернутыми конструкциями без этих чертовых сишных вывертов.
Но в итоге его куски я просто стараюсь не трогать без особой нужды, иначе потом очень трудно вклеивать изменения из новых версий.

diff -= (1 << len) - !shl;

Да я уже привык к такому стилю опенсорса. Стараюсь не трогать его, если нельзя взять и использовать как есть, либо обойтись небольшими точечными изменениями, а писать сам все.

Не удивила, а скорее умилила попытка Dave Coffin усидеть на двух стульях: заставить dcraw работать как с lcms, так и без неё. Вылилось это в 2 не пересекающихся пути преобразования цветовых пространств: convert_to_rgb(), использующая матрицу rgb_cam (считается в adobe_coeff()); и "CMS-way", - средствами lcms (apply_profile()), используется, если явно указан профиль (хотя бы входной).

Имхо, всё можно было бы упростить, если использовать для этого только lcms с генерацией профиля "на лету", по таблицам, если он не задан явно...