website icon indicating copy to clipboard operation
website copied to clipboard

new channels article

Open ringabout opened this issue 4 years ago • 8 comments

ringabout avatar Mar 11 '21 01:03 ringabout

@ee7 can you proof-read please? :P

ghost avatar Mar 12 '21 14:03 ghost

I added some suggestions but bear in mind that I'm not a native English speaker :)

ghost avatar Mar 12 '21 14:03 ghost

@Yardanico thanks for your advice!

ringabout avatar Mar 12 '21 14:03 ringabout

thanks @ee7

ringabout avatar Mar 14 '21 13:03 ringabout

I see that @narimiran changed the date - does that mean this is going to be posted soon?

This article currently advertises "safe to use" quite a lot. But did we fix enough bugs in std/channels? Maybe we should use a more conservative wording if we're not fully confident yet. See e.g. https://github.com/nim-lang/Nim/issues/17392

ee7 avatar May 03 '21 06:05 ee7

The new channels library is efficient and safe to use

Why even say this? It implies that it has a reputation as not being safe to use. Just omit this wording completely, the assumption is that libraries are safe to use when they are being advertised as "ready for use".

dom96 avatar May 03 '21 13:05 dom96

Since the channel module may be rewritten/redesigned, this article will become a bit irrelevant. So I mark it as a draft.

ringabout avatar Jul 12 '21 10:07 ringabout

Since the channel module may be rewritten/redesigned, this article will become a bit irrelevant. So I mark it as a draft.

Yes, I am in process of re-writing the module as the old one has a lot of problems including one or two data races. It's almost finished. When it has been accepted, I'll revisit this to see if this document reflects what the actual module has become.

GordonBGood avatar Jul 24 '21 03:07 GordonBGood