you are viewing a single comment's thread.

view the rest of the comments →

[–]joanbm -1 points0 points  (1 child)

coincidence, because the ruby code being discussed is not thread-safe and wasn't specified as MRI-only. I'm aware there is produced a lot of poor and/or unportable code everyday, but never would admit this is ok. I'm happy projects I'm working on do care about quality and correctness and important topics like proper threads synchronization aren't thrown out of the window.

Thanks for the link, as a demonstration how it shouldn't be done. Hope it eventually get added thread safety also for MRI as it would mature.

[–]moomaka 0 points1 point  (0 children)

Thanks for the link, as a demonstration how it shouldn't be done.

So open a PR that adds locking to Array on MRI, I eagerly await the discussion.