Skip to content

Do not embed mutex

Mikhail Mazurskiy requested to merge (removed):dont-embed-mutex into master

What does this MR do?

Makes sync.Mutex a field rather than an embedded type.

Why was this MR needed?

Cleanup to follow best practice. Embedding exposes methods of the embedded type. Mutex is an implementation detail and it's methods should not be exposed.

Are there points in the code the reviewer needs to double check?

No.

Does this MR meet the acceptance criteria?

N/A

What are the relevant issue numbers?

None.

Merge request reports