tree 10c48a8544d51328163487c046edcaaff4db74a6
parent af9dce0c6538931ecd4efa369df57e7af9061c0e
author Todd Wang <toddw@google.com> 1418409026 -0800
committer Todd Wang <toddw@google.com> 1418409032 -0800

veyron2/vdl: Fix race in TypeFromReflect.

The race would cause TypeBuilder to be called with two different
pointers for the same underlying type, which results in the
"duplicate type names" error.

FYI I tried to add a localized test to expose the race, but was
unsuccessful.  Note that there's already a multi-goroutine test
in TestTypeFromReflect.

This fixes bug #585, where TestDebugServer would reliably fail
within ~100 runs without the fix, and passes 1000 runs with the
fix.

Change-Id: I1bd3dc6433a52fd3f9d9acc12f61dddd11ffcc88