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 лет совершенно бесполезный код продолжал работать, работать и работать. :) В реальности он немного более наворочанный, возможно это его и спасало все это время.
0 коммент.:
Отправить комментарий