search

huff-language / huffmate

A library of modern, hyper-optimized, and extensible Huff contracts with extensive testing and documentation built by Huff maintainers.
https://github.com/pentagonxyz/huffmate
MIT License
437 stars 55 forks source link

fix: Dynamic Constructor Args #63

Closed refcell closed 1 year ago

refcell commented 2 years ago

Overview

Tokens currently don't use dynamic constructor arguments, but rather static strs.

MathisGD commented 1 year ago

Do you mean that symbol and name should be stored in storage ? This would mean that they would be more expensive to get, whereas constant could be set at compilation (https://github.com/huff-language/huff-rs/pull/171).

yorhodes commented 1 year ago

@MathisGD my guess is "immutables" eg constructor arguments "stored" in bytecode

refcell commented 1 year ago

Implemented by the 🐐 @MathisGD in #89