Merge pull request #3403 from VoR0220/fixedPointsAbi
accounts/abi: prepare ABI to handle fixed point types
This commit is contained in:
		@@ -33,7 +33,7 @@ const (
 | 
				
			|||||||
	FixedBytesTy
 | 
						FixedBytesTy
 | 
				
			||||||
	BytesTy
 | 
						BytesTy
 | 
				
			||||||
	HashTy
 | 
						HashTy
 | 
				
			||||||
	RealTy
 | 
						FixedpointTy
 | 
				
			||||||
)
 | 
					)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
// Type is the reflection of the supported argument type
 | 
					// Type is the reflection of the supported argument type
 | 
				
			||||||
@@ -57,16 +57,16 @@ var (
 | 
				
			|||||||
	// Types can be in the format of:
 | 
						// Types can be in the format of:
 | 
				
			||||||
	//
 | 
						//
 | 
				
			||||||
	// 	Input  = Type [ "[" [ Number ] "]" ] Name .
 | 
						// 	Input  = Type [ "[" [ Number ] "]" ] Name .
 | 
				
			||||||
	// 	Type   = [ "u" ] "int" [ Number ] .
 | 
						// 	Type   = [ "u" ] "int" [ Number ] [ x ] [ Number ].
 | 
				
			||||||
	//
 | 
						//
 | 
				
			||||||
	// Examples:
 | 
						// Examples:
 | 
				
			||||||
	//
 | 
						//
 | 
				
			||||||
	//      string     int       uint       real
 | 
						//      string     int       uint       fixed
 | 
				
			||||||
	//      string32   int8      uint8      uint[]
 | 
						//      string32   int8      uint8      uint[]
 | 
				
			||||||
	//      address    int256    uint256    real[2]
 | 
						//      address    int256    uint256    fixed128x128[2]
 | 
				
			||||||
	fullTypeRegex = regexp.MustCompile("([a-zA-Z0-9]+)(\\[([0-9]*)?\\])?")
 | 
						fullTypeRegex = regexp.MustCompile("([a-zA-Z0-9]+)(\\[([0-9]*)\\])?")
 | 
				
			||||||
	// typeRegex parses the abi sub types
 | 
						// typeRegex parses the abi sub types
 | 
				
			||||||
	typeRegex = regexp.MustCompile("([a-zA-Z]+)([0-9]*)?")
 | 
						typeRegex = regexp.MustCompile("([a-zA-Z]+)(([0-9]+)(x([0-9]+))?)?")
 | 
				
			||||||
)
 | 
					)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
// NewType creates a new reflection type of abi type given in t.
 | 
					// NewType creates a new reflection type of abi type given in t.
 | 
				
			||||||
@@ -97,7 +97,7 @@ func NewType(t string) (typ Type, err error) {
 | 
				
			|||||||
	parsedType := typeRegex.FindAllStringSubmatch(res[1], -1)[0]
 | 
						parsedType := typeRegex.FindAllStringSubmatch(res[1], -1)[0]
 | 
				
			||||||
	// varSize is the size of the variable
 | 
						// varSize is the size of the variable
 | 
				
			||||||
	var varSize int
 | 
						var varSize int
 | 
				
			||||||
	if len(parsedType[2]) > 0 {
 | 
						if len(parsedType[3]) > 0 {
 | 
				
			||||||
		var err error
 | 
							var err error
 | 
				
			||||||
		varSize, err = strconv.Atoi(parsedType[2])
 | 
							varSize, err = strconv.Atoi(parsedType[2])
 | 
				
			||||||
		if err != nil {
 | 
							if err != nil {
 | 
				
			||||||
 
 | 
				
			|||||||
							
								
								
									
										78
									
								
								accounts/abi/type_test.go
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										78
									
								
								accounts/abi/type_test.go
									
									
									
									
									
										Normal file
									
								
							@@ -0,0 +1,78 @@
 | 
				
			|||||||
 | 
					// Copyright 2016 The go-ethereum Authors
 | 
				
			||||||
 | 
					// This file is part of the go-ethereum library.
 | 
				
			||||||
 | 
					//
 | 
				
			||||||
 | 
					// The go-ethereum library is free software: you can redistribute it and/or modify
 | 
				
			||||||
 | 
					// it under the terms of the GNU Lesser General Public License as published by
 | 
				
			||||||
 | 
					// the Free Software Foundation, either version 3 of the License, or
 | 
				
			||||||
 | 
					// (at your option) any later version.
 | 
				
			||||||
 | 
					//
 | 
				
			||||||
 | 
					// The go-ethereum library is distributed in the hope that it will be useful,
 | 
				
			||||||
 | 
					// but WITHOUT ANY WARRANTY; without even the implied warranty of
 | 
				
			||||||
 | 
					// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 | 
				
			||||||
 | 
					// GNU Lesser General Public License for more details.
 | 
				
			||||||
 | 
					//
 | 
				
			||||||
 | 
					// You should have received a copy of the GNU Lesser General Public License
 | 
				
			||||||
 | 
					// along with the go-ethereum library. If not, see <http://www.gnu.org/licenses/>.
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					package abi
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					import (
 | 
				
			||||||
 | 
						"reflect"
 | 
				
			||||||
 | 
						"testing"
 | 
				
			||||||
 | 
					)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					// typeWithoutStringer is a alias for the Type type which simply doesn't implement
 | 
				
			||||||
 | 
					// the stringer interface to allow printing type details in the tests below.
 | 
				
			||||||
 | 
					type typeWithoutStringer Type
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					// Tests that all allowed types get recognized by the type parser.
 | 
				
			||||||
 | 
					func TestTypeRegexp(t *testing.T) {
 | 
				
			||||||
 | 
						tests := []struct {
 | 
				
			||||||
 | 
							blob string
 | 
				
			||||||
 | 
							kind Type
 | 
				
			||||||
 | 
						}{
 | 
				
			||||||
 | 
							{"int", Type{Kind: reflect.Ptr, Type: big_t, Size: 256, T: IntTy, stringKind: "int256"}},
 | 
				
			||||||
 | 
							{"int8", Type{Kind: reflect.Int8, Type: big_t, Size: 8, T: IntTy, stringKind: "int8"}},
 | 
				
			||||||
 | 
							{"int256", Type{Kind: reflect.Ptr, Type: big_t, Size: 256, T: IntTy, stringKind: "int256"}},
 | 
				
			||||||
 | 
							{"int[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Ptr, Type: big_t, Size: 256, T: IntTy, stringKind: "int256"}, stringKind: "int256[]"}},
 | 
				
			||||||
 | 
							{"int[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{Kind: reflect.Ptr, Type: big_t, Size: 256, T: IntTy, stringKind: "int256"}, stringKind: "int256[2]"}},
 | 
				
			||||||
 | 
							{"int32[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Int32, Type: big_t, Size: 32, T: IntTy, stringKind: "int32"}, stringKind: "int32[]"}},
 | 
				
			||||||
 | 
							{"int32[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{Kind: reflect.Int32, Type: big_t, Size: 32, T: IntTy, stringKind: "int32"}, stringKind: "int32[2]"}},
 | 
				
			||||||
 | 
							{"uint", Type{Kind: reflect.Ptr, Type: ubig_t, Size: 256, T: UintTy, stringKind: "uint256"}},
 | 
				
			||||||
 | 
							{"uint8", Type{Kind: reflect.Uint8, Type: ubig_t, Size: 8, T: UintTy, stringKind: "uint8"}},
 | 
				
			||||||
 | 
							{"uint256", Type{Kind: reflect.Ptr, Type: ubig_t, Size: 256, T: UintTy, stringKind: "uint256"}},
 | 
				
			||||||
 | 
							{"uint[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Ptr, Type: ubig_t, Size: 256, T: UintTy, stringKind: "uint256"}, stringKind: "uint256[]"}},
 | 
				
			||||||
 | 
							{"uint[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{Kind: reflect.Ptr, Type: ubig_t, Size: 256, T: UintTy, stringKind: "uint256"}, stringKind: "uint256[2]"}},
 | 
				
			||||||
 | 
							{"uint32[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Uint32, Type: big_t, Size: 32, T: UintTy, stringKind: "uint32"}, stringKind: "uint32[]"}},
 | 
				
			||||||
 | 
							{"uint32[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{Kind: reflect.Uint32, Type: big_t, Size: 32, T: UintTy, stringKind: "uint32"}, stringKind: "uint32[2]"}},
 | 
				
			||||||
 | 
							{"bytes", Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Uint8, Type: ubig_t, Size: 8, T: UintTy, stringKind: "uint8"}, T: BytesTy, stringKind: "bytes"}},
 | 
				
			||||||
 | 
							{"bytes32", Type{IsArray: true, SliceSize: 32, Elem: &Type{Kind: reflect.Uint8, Type: ubig_t, Size: 8, T: UintTy, stringKind: "uint8"}, T: FixedBytesTy, stringKind: "bytes32"}},
 | 
				
			||||||
 | 
							{"bytes[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Uint8, Type: ubig_t, Size: 8, T: UintTy, stringKind: "uint8"}, T: BytesTy, stringKind: "bytes"}, stringKind: "bytes[]"}},
 | 
				
			||||||
 | 
							{"bytes[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Uint8, Type: ubig_t, Size: 8, T: UintTy, stringKind: "uint8"}, T: BytesTy, stringKind: "bytes"}, stringKind: "bytes[2]"}},
 | 
				
			||||||
 | 
							{"bytes32[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{IsArray: true, SliceSize: 32, Elem: &Type{Kind: reflect.Uint8, Type: ubig_t, Size: 8, T: UintTy, stringKind: "uint8"}, T: FixedBytesTy, stringKind: "bytes32"}, stringKind: "bytes32[]"}},
 | 
				
			||||||
 | 
							{"bytes32[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{IsArray: true, SliceSize: 32, Elem: &Type{Kind: reflect.Uint8, Type: ubig_t, Size: 8, T: UintTy, stringKind: "uint8"}, T: FixedBytesTy, stringKind: "bytes32"}, stringKind: "bytes32[2]"}},
 | 
				
			||||||
 | 
							{"string", Type{Kind: reflect.String, Size: -1, T: StringTy, stringKind: "string"}},
 | 
				
			||||||
 | 
							{"string[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.String, Size: -1, T: StringTy, stringKind: "string"}, stringKind: "string[]"}},
 | 
				
			||||||
 | 
							{"string[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{Kind: reflect.String, Size: -1, T: StringTy, stringKind: "string"}, stringKind: "string[2]"}},
 | 
				
			||||||
 | 
							{"address", Type{Kind: reflect.Array, Type: address_t, Size: 20, T: AddressTy, stringKind: "address"}},
 | 
				
			||||||
 | 
							{"address[]", Type{IsSlice: true, SliceSize: -1, Elem: &Type{Kind: reflect.Array, Type: address_t, Size: 20, T: AddressTy, stringKind: "address"}, stringKind: "address[]"}},
 | 
				
			||||||
 | 
							{"address[2]", Type{IsArray: true, SliceSize: 2, Elem: &Type{Kind: reflect.Array, Type: address_t, Size: 20, T: AddressTy, stringKind: "address"}, stringKind: "address[2]"}},
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
							// TODO when fixed types are implemented properly
 | 
				
			||||||
 | 
							// {"fixed", Type{}},
 | 
				
			||||||
 | 
							// {"fixed128x128", Type{}},
 | 
				
			||||||
 | 
							// {"fixed[]", Type{}},
 | 
				
			||||||
 | 
							// {"fixed[2]", Type{}},
 | 
				
			||||||
 | 
							// {"fixed128x128[]", Type{}},
 | 
				
			||||||
 | 
							// {"fixed128x128[2]", Type{}},
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
						for i, tt := range tests {
 | 
				
			||||||
 | 
							typ, err := NewType(tt.blob)
 | 
				
			||||||
 | 
							if err != nil {
 | 
				
			||||||
 | 
								t.Errorf("type %d: failed to parse type string: %v", i, err)
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
							if !reflect.DeepEqual(typ, tt.kind) {
 | 
				
			||||||
 | 
								t.Errorf("type %d: parsed type mismatch:\n  have %+v\n  want %+v", i, typeWithoutStringer(typ), typeWithoutStringer(tt.kind))
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
		Reference in New Issue
	
	Block a user