Optimizations to script builder

[WyattMufson]

You don't need to call pushData on a String since it will just end up calling pushHexString. This change just makes it a little clearer how the ScriptBuilder works.

[WyattMufson]

Also it looks like all of the non-typed portions of the script builder can be completely removed. pushContractInvoke is only being called by buildNEP5TransferScript which is not being used anywhere. This will allow for removing a large chunk of the script builder (I'm doing the same thing for neovm-utils)

[saltyskip]

if we change this can we remove pushData, buildNep5TransferScript, and pushContractInvoke from the project as well in this PR

[WyattMufson]

Sure thing. Are you only using the typed functions now?

[saltyskip]

Yes i believe only typed functions are necessary as every contract invoke should filter in as a DAPI request structure.

[WyattMufson]

I'm talking with Apisit about how to fix

OzoneWalletIOS/O3/New Group/Clients/Wallet.swift

Line 529 in decf61d

let scriptBytes = [UInt8(script.count)] + script

where script.count is bigger than 255.

[WyattMufson]

@saltyskip removed the rest. Didn't find a solution for when script.count is too big for UInt8