You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

James Bardin 657932261b Make sure shadow.closeWalker doesn't copy Mutexes ... The Close methods on shadow.Values require pointer receivers because they contain a sync.Mutex, but that value was being copied through Value.Interface by the closeWalker. Because reflectwalk passes the struct fields to the StructField method as they are defined in the struct, and they may have been read as a value, we can't immediately call Interface() to check the method set without possibly copying the internal mutex values. Use the Implements method to first check if we need to call Interface, and if it's not, then we can check if the value is addressable. Because of this use of reflection, we can't vet for the copying of these locks. The minimal amount of code in the Close method left us only with a race detected within the mutex itself, which leads to a stacktrace pointing to the runtime rather than our code.		9 years ago
..
closer.go	Make sure shadow.closeWalker doesn't copy Mutexes	9 years ago
closer_test.go	helper/shadow: Close for auto-closing all values	9 years ago
compared_value.go	helper/shadow: ComparedValue	10 years ago
compared_value_test.go	helper/shadow: ComparedValue	10 years ago
keyed_value.go	helper/shadow: keyedValue.WaitForChange must unlock	9 years ago
keyed_value_test.go	helper/shadow: keyedValue.WaitForChange must unlock	9 years ago
ordered_value.go	…
ordered_value_test.go	…
value.go	make shadow.Value a Locker	9 years ago
value_test.go	helper/shadow: KeyedValue.Close	10 years ago