Question

I have some legacy code that uses Interlocked.Equals to compare values. The values may be two bools or it may compare an array of structs to null. Resharper complains about Interlocked.Equals saying “access to a static member of a type via a derived type”. I know that Equals is not a member of the Interlocked class, but rather is a member of the object class. The comparison is happening in a thread so I assume the original coder wanted to do the comparison as an atomic operation hence the use of Interlocked. Since object.Equals is not atomic what is the proper, thread safe, way to do these kinds of comparisons?
Note, most of the data is static, some of it is static volatile.

Answer 1

Single reads of booleans or object references are atomic. So, if you’re comparing one shared value to a constant or local variable, no “interlocking” is necessary. As Jon stated, you would need to use Interlocked.CompareExchange to ensure you’re reading the latest value written, unless the shared variables are volatile.

If both comparands are shared, then you’ll need an actual lock. There isn’t a way to atomically compare two shared values AFAIK.

Update:

I do recommend the introduction of explicit locks for shared data. Keep in mind that the original code was completely broken, so don’t be shy about changing it.

If you think about it, what would you do with the result of the comparison? It doesn’t really make sense; as soon as you have the result, it could be wrong. A lock would need to be held longer than just for the comparision for the comparision to be useful for anything.