вторник, 30 августа 2011 г.

Давным давно, в одном редко вызываемом модуле...

Нет, все действия происходят в настоящее время. Наткнулся на интересный кусок кода:
	boost::format value("%s:%s");

value % head;

BOOST_FOREACH(value_type &item, container) {
if (item.valid()) {
value % item.value();
}
}

return 0;
}
Все совпадения с реальным кодом следует считать совпадениями, конечно я его немного поменял. :)

Наткнулся потому, что из за него повалились вдруг тесты - мало аргументов для boost::format.

Долго думал, пока наконец не понял, что этот код совершенно никому не нужен. value, объявленная в начале фрагмента благополучно уничтожается в конце оного.

Полез в историю, и выяснил, что в мае 2006 году объявление value находилось в другом месте, в начале модуля. И действительно использовалось в другом месте.

Через два месяца, в июле 2006, код, который использовал это значение был переписан. значение осталось без использования.

Через месяц определение value было перенесено в начало вышеуказанного фрагмента. В этот момент уже можно было оценить всю бесполезность кода, но почему-то этого не произошло, даже более того...

В новейшей истории, в этом месяце, код был переписан с использованием BOOST_FOREACH.

Пока наконец сегодня не сломались тесты, и этот код не был выброшен на свалку истории.

Забавно, что почти 6 лет совершенно бесполезный код продолжал работать, работать и работать. :) В реальности он немного более наворочанный, возможно это его и спасало все это время.