clearmatics / mobius

Trustless Tumbling for Transaction Privacy
GNU Lesser General Public License v3.0
86 stars 23 forks source link

Solidity best practice #47

Closed zoenolan closed 6 years ago

zoenolan commented 6 years ago

We should be consistent either constant or pure/view on functions. Seems bad style to mix the two

HarryR commented 6 years ago

The only thing that really matters is appropriately using the pure keyword when the compiler supports it, hence #48.