```suggestion deprecated: "`deinitRLock` is not needed anymore in ARC/ORC (it is a no-op now); `=destroy` is already defined for `RLock`".} = ``` It would have been easier to just commit my proposed changes ;)

```suggestion ## Nim lock, re-entrant. ```

```suggestion deprecated: "`deinitCond` is not needed anymore in ARC/ORC (it is a no-op now); `=destroy` is already defined for `Cond`".} = ```

```suggestion deprecated: "`deinitLock` is not needed anymore in ARC/ORC (it is a no-op now); `=destroy` is already defined for `Lock`".} = ```

```suggestion ## Nim condition variable. ```

Should tests also test ORC?

```suggestion deprecated: "`deinitRLock` is not needed anymore in ARC/ORC (it is a no-op now); `=destroy` is already defined for `RLock`".} = ```

```suggestion RLock* = object ## Nim lock, re-entrant. ```

```suggestion Cond* = object ## Nim condition variable. ```

```suggestion Lock* = object ## Nim lock; whether this is re-entrant or not is unspecified! ```

```suggestion RLock* = object ## Nim lock, re-entrant. lock: SysLock ```

```suggestion deprecated: "`deinitCond` is not needed anymore in ARC/ORC (it is a no-op now); `=destroy` is already defined for `Cond`".} = ```

```suggestion deprecated: "`deinitLock` is not needed anymore in ARC/ORC (it is a no-op now); `=destroy` is already defined for `Lock`".} = ```

```suggestion Cond* = object ## Nim condition variable. cond: SysCond ``` We want the doc comment to be attached to the type.

```suggestion Lock* = object ## Nim lock; whether this is re-entrant or not is unspecified! lock: SysLock ``` Otherwise the doc comment is attached to the field.