-
Notifications
You must be signed in to change notification settings - Fork 677
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
gopl-zh ch8问题反馈 #14
Comments
8.7 的gopl.io/ch8/countdown1例程中的 |
@bluebanboom ,已修正~ |
8.4 正文第三段:因此调用者何(和)被调用者将引用同一个channel对象 |
@dreamrover ,感谢! |
又发现一些: 8.4.1节 倒数第三段: 8.4.2节: 8.4.3节: 8.4.4节: |
8.5节: 正文第五段:没(每)一个文件名对应一个 makeThumbnails5代码之后第二段:这个计数器需要在多个goroutine操作时做到安全并且提供提供在其减为零之前一直等待的一种方法——多了一个“提供” 正文倒数第二段:
——没表达清楚,这样翻译应该更好:
|
@dreamrover ,本章已修正~ |
8.4.2最后一段 "视图重复关闭....... , 视图关闭" 应该为试图 |
@jau1jz,你看的不是新版 |
8.6 主函数和5.6节中的breadthFirst(深度优先)类似。 -->广度优先 |
@stepdc ,thx。。已修正,这个译错是我的,好低级orz |
ch8-04-2,倒数第二段,第二句:“只要”应该改为“只有”才通顺,请斟酌,谢谢! |
@klifish ,thx,已修正 |
就直接提了: 8.5 8.6 这里的n是fd的limit-20 另外,关于long-lived的翻译,借用了一下这里的讨论,译为“常驻”颇为合适。 |
@klifish , thx! 已修正 |
标题 |
@bianchui , thx, fixed |
@zengming00 ,只要没有在监听 ch 的 goroutine 就没事,不过技术问题还是找专门的论坛讨论比较好,这里主要是反映书里的错误的~ |
ch为空没问题,runtime会回收资源,最多是goroutine会浪费一些cpu。如果不想浪费可以试试context包。话说代码简单也是很有意义的,不要太吝啬cpu,除非这种错误经常发生 |
@zengming00 对于这种不影响大局的问题,指望别人改是不靠谱的。如果你觉得确实有注释的必要,可以自己尝试去提交一个PR,这样被接受的机率会大很多 |
@zengming00 ,我的意思是去技术社区讨论可以把内容沉淀下来,比如gocn.io,不过也确实如柴大所说,如果你觉得有问题,提一个 pr 补一个译注也挺好的,我们看没问题的话会给合并 另外我们这里毕竟只是翻译,补充一些译注也是依照译者的理解和风格去加,如果原作者没有写的东西。。指望译者都能补全有点难啊。。 另外,就这里说的这个技术问题,函数退出之后 chan 不排空问题也不大,之后 chan 会被 GC 掉,只要没有 goroutine 在监听就可以 |
8.4.1 Unbuffered Channels 第二段最后一句: reawakening一词翻译为唤醒好像不太准确,“再次唤醒/重启”可能好一些,毕竟是reawakening不是awakening,而且会让人费解,唤醒发送者goroutine之前接收者为什么会收到数据?考虑到对无缓存channel,相应的接收在重新唤醒发送goroutine之前完成应该合理。不知道我的理解是否正确。 |
@liuz430524,按你的建议修改了一下,再次唤醒 |
8.4.2 小节倒数第二段 “不使用的使用”是否应该改为“不使用的时候”? |
@wddpct ,你看的是老版本,仓库中的是比较新的了 |
@cch123 我看的是gitbook版本,不过谢谢了 |
RT
有问题在这里提出,会尽快修正
The text was updated successfully, but these errors were encountered: