История изменений
Исправление
emorozov,
(текущая версия)
:
Да, спасибо, я уже понял, что у меня неполная версия. Поищу полную.
И в Smacker плохо реализовано управление памятью. Вангую, что оно наверняка течёт во многих местах, ибо по той же обработке ошибок видно, что логика освобождения памяти не продумывалась совершенно, и не тестировалось, что происходит в случае малейшей ошибки.
Всё рассчитано на happy path - что нигде не случится ни единого отклонения от единственного пути, который предусмотрел разработчик.
По коду было непонятно, что это сторонний проект. Просто замечание: как правило, когда сторонний проект подключается копированием кода, а не другими способами, традиционно копии кладутся в каталог с названием типа vendor
, vendorized
, и т.п., чтобы явно пометить, что это скопированные зависимости третьей стороны, а не код проекта.
И по логу git тоже было непонятно, что это копия.
Исправление
emorozov,
:
Да, спасибо, я уже понял, что у меня неполная версия. Поищу полную.
И в Smacker плохо реализовано управление памятью. Вангую, что оно наверняка течёт во многих местах, ибо по той же обработке ошибок видно, что логика освобождения памяти не продумывалась совершенно, и не тестировалось, что происходит в случае малейшей ошибки.
Всё рассчитано на happy path - что нигде не случится ни единого отклонения от единственного пути, который предусмотрел разработчик.
По коду было непонятно, что это сторонний проект. Просто замечание: как правило, когда сторонний проект подключается копированием кода, а не другими способами, традиционно копии кладутся в каталог с названием типа vendor
, vendorized
, и т.п., чтобы явно пометить, что это скопированные зависимости третьей стороны, а не код проекта.
Исходная версия
emorozov,
:
Да, спасибо, я уже понял, что у меня неполная версия. Поищу полную.
И в Smacker плохо реализовано управление памяти. Вангую, что оно наверняка течёт во многих местах, ибо по той же обработке ошибок видно, что логика освобождения памяти не продумывалась совершенно, и не тестировалось, что происходит в случае малейшей ошибки.
Всё рассчитано на happy path - что нигде не случится ни единого отклонения от единственного пути, который предусмотрел разработчик.
По коду было непонятно, что это сторонний проект. Просто замечание: как правило, когда сторонний проект подключается копированием кода, а не другими способами, традиционно копии кладутся в каталог с названием типа vendor
, vendorized
, и т.п., чтобы явно пометить, что это скопированные зависимости третьей стороны, а не код проекта.