Обновить

Комментарии 2

Выглядит интересно, но так как это библиотека, то есть вопрос: почему жёстко фиксируется solidity версии 0.8.33? В коде не увидел использования никаких фичей из новых версий, однако столь высокая версия ограничивает использование на многих сетях, где ещё не ввели новые опкоды (например, Moonbeam, Merlinchain etc). ИМХО при разработке библиотек лучше всего всегда указывать минимально возможную версию solidity с прагмой >=, чтобы всё компилировалось в как можно большем числе проектов на максимально возможном числе сетей.

с одной стороны, этот код я скопировал из своего другого проекта, где использую transient, а это как раз 0.8.33, емпин; (там можно было так жестко указать версию, без последствий, да и я предпочитаю жестко задавать версии, чтобы не было потом "случайных" обновлений)
с другой стороны, я так-то забил на эту деталь... Вообще, хороший поинт, я пересмотрю код. Думаю, 0.8.0 должно хватить (из breaking changes после 0.8.0 я помню только `assembly("memory-safe")` в 0.8.13, но это не используется в данной библиотечке)

Зарегистрируйтесь на Хабре, чтобы оставить комментарий

Публикации