Reporte de Adri

[eugenioclrc]

• el valid token id en huff quizas se puede sacar, ya lo garantiza el contrato que mintea que esta en el rango 1-1024 #5

valid, me gustaba para dar garantias y ademas meter mas cositas en huff, es redundante y puede sacarse, nobleza obliga, magnetto me tiro un dm por esto mismo

• si no se puede sacar, una optimizacion menor en vez de hacer tokenId != 0 && tokenId < 1025 es directamente hacer tokenId - 1 < 1024 (si es 0 hacer overflow a max int y da bien igual)

prefiro sacar el anterior

• en tokenURI() fijate que el calldata que armar para llamar al token rendered es exactamente el mismo que tu function (mismo sig y argumentos), podrias hacer directamente calldatacopy y mandar eso #3

me gusta

• sacar el import console en el contrato de huff

quedo por si tenia que debuggear algo, esta bueno sacarlo, pero por las dudas aclaro que si no se usa no se transpila a bytecode

• en huff podes usar __RIGHTPAD para los literales de strings tipo el UwU para evitar ese choclo de ceros #6

+1

• en huff algunas funciones tienen non-payable y otras no, por que? ninguna deberia recibir eth #4

ninguna deberia recibir ether, pero hacer el chequeo non_payable es un chequeo extra de gas, no tengo respuesta para esto mas que hay que revisarlas

• para que sirve totalSupply()? es parte del standard de enumerable pero las otras funciones no estan

lo dejamos para que quede claro el totalSupply de la coleccion, tengo entendido que etherscan lo muestra y queda lindo :D

[eugenioclrc]

All solved