sync: outdated referenced documentation in `Cond` about go HOT 3 CLOSED

In my humble opinion, I like seeing the reference to gophercon talks in the source code because it enhances the documentation, which is often lacking, and even if the std lib has changed since the date of the talk, the talk is still useful and I'm hardly misled by it.

from go.

@golang/runtime per owners -- any opinions on this issue?

from go.

I looked over the two links your pointed out, and I think it's OK to leave it as-is.

• Go advanced concurrency patterns: part 2 (timers) discusses issues with time.Timer that have been resolved in version 1.23, and possibly even earlier versions.

This one links to the upstream bugs at the very top and they're now closed. That's at least one signal to the reader that the post may not be up-to-date.

And although it contains some outdated information, the patterns are still useful to those using older versions of Go, and to be clear, the patterns will still work, so the information isn't all that detrimental. I think maybe there's something to be said here, but someone clicking through to here through sync.Cond is not going to be super likely to click the timers one.

Though, perhaps we could just politely ask @empijei to update the blog post once Go 1.23 is released. :)

• Go advanced concurrency patterns: part 3 (channels) explains how TryLock and TryRLock can be implemented, which have since become part of the standard library.

It does explain how TryLock and TryRLock can be implemented, but my impression from that page is it's much more about the expressive power of channels as an abstraction, as opposed to how to implement a missing standard library feature. This is referenced all over the blog post, but at the very top we see "The examples will correctly express the primitives but might not be as fast as the real ones." (Emphasis mine.) I think the author is clearly implying that these features may or may not already exist.

from go.