Arachnid / solidity-stringutils

Basic string utilities for Solidity
Apache License 2.0
1.12k stars 369 forks source link

Add default data location (memory) #30

Closed axic closed 5 years ago

axic commented 5 years ago

This will be explcitily required in 0.5.0. Original change by Chase McDermott.

axic commented 5 years ago

@Arachnid would appreciate if this could be merged (as a result we could merge 2 PRs in Solidity 😉)

Arachnid commented 5 years ago

Stringutils is in the solidity repo? That's terrifying.

axic commented 5 years ago

It is used in CI for compilation tests. Making sure we do not break things too often.

If ENS has a complex enough repo with truffle tests and is reasonable stable we'd be happy to include that too.