veyron: Collection of bugfixes and features for vom2 transition.
This includes:
o Change reserved.Signature / MethodSignature to allow passing
in an explicit client.
o Add vdl.Type.Walk to visit all types in the type graph, and
use it to implement Type.ContainsKind and Type.ContainsType.
o Fix VDL->Go codegen to generate zero arrays and structs with
inline typeobject and oneof values correctly.
o Fix valconv to create proper zero values for arrays, structs
and oneof, since the VDL zero semantics are different than Go
o Add some more tests (incomplete)
This collection of changes allows the javascript-integration
tests to run without panic in vom2 mode.
Change-Id: I8470597f2467c36441684f520e146cb88f985c4a
diff --git a/runtimes/google/ipc/server.go b/runtimes/google/ipc/server.go
index 1516527..af3a370 100644
--- a/runtimes/google/ipc/server.go
+++ b/runtimes/google/ipc/server.go
@@ -429,7 +429,7 @@
for {
flow, err := ln.Accept()
if err != nil {
- vlog.VI(10).Infof("ipc: Accept on %v failed: %v", ln, err)
+ vlog.VI(10).Infof("ipc: Accept on %v failed: %v", ep, err)
return
}
calls.Add(1)
@@ -437,13 +437,13 @@
defer calls.Done()
fs, err := newFlowServer(flow, s)
if err != nil {
- vlog.Errorf("newFlowServer on %v failed: %v", ln, err)
+ vlog.Errorf("newFlowServer on %v failed: %v", ep, err)
return
}
if err := fs.serve(); err != nil {
// TODO(caprita): Logging errors here is too spammy. For example, "not
// authorized" errors shouldn't be logged as server errors.
- vlog.Errorf("Flow serve on %v failed: %v", ln, err)
+ vlog.Errorf("Flow serve on %v failed: %v", ep, err)
}
}(flow)
}
diff --git a/runtimes/google/ipc/signature_test.go b/runtimes/google/ipc/signature_test.go
index 83d9c78..0f5c5ac 100644
--- a/runtimes/google/ipc/signature_test.go
+++ b/runtimes/google/ipc/signature_test.go
@@ -100,7 +100,7 @@
}},
}
for _, test := range tests {
- sig, err := reserved.MethodSignature(runtime.NewContext(), name, test.Method)
+ sig, err := reserved.MethodSignature(runtime.NewContext(), nil, name, test.Method)
if err != nil {
t.Errorf("call failed: %v", err)
}
@@ -155,7 +155,7 @@
},
}
- sig, err := reserved.Signature(runtime.NewContext(), name)
+ sig, err := reserved.Signature(runtime.NewContext(), nil, name)
if err != nil {
t.Errorf("call failed: %v", err)
}
