neo-devpack-dotnet icon indicating copy to clipboard operation
neo-devpack-dotnet copied to clipboard

Shouldn't mix static and instance methods in TokenContract and subclasses

Open devhawk opened this issue 4 years ago • 1 comments

TokenContract defines abstract instance methods Symbol and Decimals, but the other methods like TotalSupply, BalanceOf as well as subclass methods such as Transfer and OwnerOf are static. It seems odd from a C# semantics perspective to mix and match like this

devhawk avatar Jun 18 '21 20:06 devhawk

Symbol and Decimals must be instance methods because they are abstract. The others are static because static methods are cheaper.

erikzhang avatar May 21 '22 02:05 erikzhang