v.io/core: another step in eliminating verror1, and making verror2.E be a struct
This CL:
- removes all uinused parts of verror1
- changes as many uses of verror2.E as possible to "error" or "verror2.Standard"
where this can be done easily
Tested with
cd release/go/src/v.io
v23 go test ./...
v23 test run vanadium-go-test
Remaining work:
Key remaining uses of verror1 include uses of
Make Makef Abortedf BadArgf BadProtocolf Existsf Internalf NoAccessf
NoExistf Convert ConvertWithDefault
in
v.io/core/veyron/lib/testutil/dispatcher.go
v.io/core/veyron/runtimes/google/ipc/reserved.go
v.io/core/veyron/runtimes/google/ipc/server.go
v.io/core/veyron/runtimes/google/ipc/stream/manager/manager.go
v.io/core/veyron/runtimes/google/ipc/stream/proxy/proxy.go
v.io/core/veyron/runtimes/google/ipc/stream/vif/vif.go
v.io/core/veyron/runtimes/google/naming/namespace/namespace.go
v.io/core/veyron2/ipc/reflect_invoker.go
v.io/core/veyron2/ipc/reflect_invoker_test.go
v.io/core/veyron2/vdl/valconv/convert_test.go
v.io/core/veyron2/verror/verror.go
v.io/core/veyron2/verror2/verror.go
v.io/core/veyron2/verror2/verror_test.go
v.io/core/veyron2/vom/binary_decoder.go
v.io/core/veyron2/vom/binary_encoder.go
v.io/core/veyron2/vom/binary_util.go
v.io/core/veyron2/vom/decoder.go
v.io/core/veyron2/vom/dump.go
v.io/core/veyron2/vom/type.go
v.io/jni/util/util.go
v.io/jni/veyron/runtimes/google/ipc/invoker.go
The last remaining use of verror2.E that's not trvial to change (at least for me :-) is in
v.io/core/veyron/runtimes/google/ipc/testutil_test.go
in makeResultPtrs() where it does:
var verr verror2.E
typ = reflect.ValueOf(&verr).Elem().Type()
Change-Id: Icb19e0389ea60d3fc625d5f229a03056e222c0ca
3 files changed
