Gas Optimizations

[code423n4]

See the markdown file with the details of this report here.

[c4-pre-sort]

dmvt marked the issue as high quality report

[c4-judge]

dmvt marked the issue as grade-a

[drgorillamd]

Thanks for the report!
Some comments:

reservedToken should be set after the transfer is made so you can save gas if the transfer reverts

this is the transfer of the project token received from the swap, this one should never revert

abi.encodePacked() is cheaper than abi.encode()

not here, compared and exact same cost (with 200 runs) - this might be true with multiple data tho (packed vs unpacked then), with the collision risk it introduces

Optimize names to save gas

Every function of the datasource/delegate will be called once per tx (ie A-B-C has same cost if reordering to C-B-A here