go-ethereum

Commit Graph

Author	SHA1	Message	Date
Felix Lange	5f7826270c	all: unify big.Int zero checks, use common/math in more places (#3716 ) * common/math: optimize PaddedBigBytes, use it more name old time/op new time/op delta PaddedBigBytes-8 71.1ns ± 5% 46.1ns ± 1% -35.15% (p=0.000 n=20+19) name old alloc/op new alloc/op delta PaddedBigBytes-8 48.0B ± 0% 32.0B ± 0% -33.33% (p=0.000 n=20+20) * all: unify big.Int zero checks Various checks were in use. This commit replaces them all with Int.Sign, which is cheaper and less code. eg templates: func before(x big.Int) bool { return x.BitLen() == 0 } func after(x big.Int) bool { return x.Sign() == 0 } func before(x big.Int) bool { return x.BitLen() > 0 } func after(x big.Int) bool { return x.Sign() != 0 } func before(x big.Int) int { return x.Cmp(common.Big0) } func after(x big.Int) int { return x.Sign() } * common/math, crypto/secp256k1: make ReadBits public in package math	8 years ago
RJ	2126d81488	accounts/abi: add support for function types (#3405 )	8 years ago
Thomas Bocek	89c6c5bb85	accounts/abi: Negative numbers not properly converted in ABI encoding When converting a negative number e.g., -2, the resulting ABI encoding should look as follows: fffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffffe. However, since the check of the type is for an uint instead of an int, it results in the following ABI encoding: 0101010101010101010101010101010101010101010101010101010101010102. The Ethereum ABI (https://github.com/ethereum/wiki/wiki/Ethereum-Contract-ABI) says, that signed integers are stored in two's complement which should be of the form ffffff.... and not 01010101..... for e.g. -1. Thus, I removed the type check in numbers.go as well as the function S256 as I don't think they are correct. Or maybe I'm missing something?	9 years ago
Jeffrey Wilcke	4880868c88	accounts/abi: fixed string and fixed size bytes packing	9 years ago
Jeffrey Wilcke	5127ec10cb	accouns/abi: refactored ABI package Refactored the abi package parsing and type handling. Relying mostly on package reflect as opposed to most of our own type reflection. Our own type reflection is still used however for cases such as Bytes and FixedBytes (abi: bytes•). This also inclused several fixes for slice handling of arbitrary and fixed size for all supported types. This also further removes implicit type casting such as assigning, for example `[2]T{} = []T{1}` will fail, however `[2]T{} == []T{1, 2}` (notice assigning slice to fixed size array). Assigning arrays to slices will always succeed if they are of the same element type. Incidentally also fixes #2379	9 years ago

5 Commits (cf19586cfbe5aa379c8fdb046dc5a8c0fa1cebbb)