mirror of https://github.com/status-im/op-geth.git
common: drop accessors for Value.Val
I don't see why we would need two different accessors for a public field.
This commit is contained in:
parent
dcb9614dfe
commit
c161d73d42
|
@ -112,7 +112,7 @@ func Encode(object interface{}) []byte {
|
|||
if object != nil {
|
||||
switch t := object.(type) {
|
||||
case *Value:
|
||||
buff.Write(Encode(t.Raw()))
|
||||
buff.Write(Encode(t.Val))
|
||||
case RlpEncodable:
|
||||
buff.Write(Encode(t.RlpData()))
|
||||
// Code dup :-/
|
||||
|
|
|
@ -57,14 +57,6 @@ func (val *Value) Len() int {
|
|||
return len(val.Bytes())
|
||||
}
|
||||
|
||||
func (val *Value) Raw() interface{} {
|
||||
return val.Val
|
||||
}
|
||||
|
||||
func (val *Value) Interface() interface{} {
|
||||
return val.Val
|
||||
}
|
||||
|
||||
func (val *Value) Uint() uint64 {
|
||||
if Val, ok := val.Val.(uint8); ok {
|
||||
return uint64(Val)
|
||||
|
|
|
@ -35,7 +35,7 @@ func (s *ValueSuite) TestValueTypes(c *checker.C) {
|
|||
|
||||
c.Assert(str.Str(), checker.Equals, strExp)
|
||||
c.Assert(num.Uint(), checker.Equals, numExp)
|
||||
c.Assert(NewValue(inter.Interface()).Cmp(NewValue(interExp)), checker.Equals, true)
|
||||
c.Assert(NewValue(inter.Val).Cmp(NewValue(interExp)), checker.Equals, true)
|
||||
c.Assert(byt.Bytes(), checker.DeepEquals, bytExp)
|
||||
c.Assert(bigInt.BigInt(), checker.DeepEquals, bigExp)
|
||||
}
|
||||
|
|
|
@ -49,7 +49,7 @@ func (db *MemDatabase) Print() {
|
|||
for key, val := range db.db {
|
||||
fmt.Printf("%x(%d): ", key, len(key))
|
||||
node := common.NewValueFromBytes(val)
|
||||
fmt.Printf("%q\n", node.Interface())
|
||||
fmt.Printf("%q\n", node.Val)
|
||||
}
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue