LINUX.ORG.RU

История изменений

Исправление emorozov, (текущая версия) :

Да, спасибо, я уже понял, что у меня неполная версия. Поищу полную.

И в Smacker плохо реализовано управление памятью. Вангую, что оно наверняка течёт во многих местах, ибо по той же обработке ошибок видно, что логика освобождения памяти не продумывалась совершенно, и не тестировалось, что происходит в случае малейшей ошибки.

Всё рассчитано на happy path - что нигде не случится ни единого отклонения от единственного пути, который предусмотрел разработчик.

По коду было непонятно, что это сторонний проект. Просто замечание: как правило, когда сторонний проект подключается копированием кода, а не другими способами, традиционно копии кладутся в каталог с названием типа vendor, vendorized, и т.п., чтобы явно пометить, что это скопированные зависимости третьей стороны, а не код проекта.

И по логу git тоже было непонятно, что это копия.

Исправление emorozov, :

Да, спасибо, я уже понял, что у меня неполная версия. Поищу полную.

И в Smacker плохо реализовано управление памятью. Вангую, что оно наверняка течёт во многих местах, ибо по той же обработке ошибок видно, что логика освобождения памяти не продумывалась совершенно, и не тестировалось, что происходит в случае малейшей ошибки.

Всё рассчитано на happy path - что нигде не случится ни единого отклонения от единственного пути, который предусмотрел разработчик.

По коду было непонятно, что это сторонний проект. Просто замечание: как правило, когда сторонний проект подключается копированием кода, а не другими способами, традиционно копии кладутся в каталог с названием типа vendor, vendorized, и т.п., чтобы явно пометить, что это скопированные зависимости третьей стороны, а не код проекта.

Исходная версия emorozov, :

Да, спасибо, я уже понял, что у меня неполная версия. Поищу полную.

И в Smacker плохо реализовано управление памяти. Вангую, что оно наверняка течёт во многих местах, ибо по той же обработке ошибок видно, что логика освобождения памяти не продумывалась совершенно, и не тестировалось, что происходит в случае малейшей ошибки.

Всё рассчитано на happy path - что нигде не случится ни единого отклонения от единственного пути, который предусмотрел разработчик.

По коду было непонятно, что это сторонний проект. Просто замечание: как правило, когда сторонний проект подключается копированием кода, а не другими способами, традиционно копии кладутся в каталог с названием типа vendor, vendorized, и т.п., чтобы явно пометить, что это скопированные зависимости третьей стороны, а не код проекта.