Revise chapter 3 #54
Merged
Revise chapter 3 #54
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fwqaaq
reviewed
Jul 10, 2023
| @@ -138,10 +138,10 @@ fn f() { | |||
| } | |||
| ``` | |||
|
|
|||
| 由于 join 和产生操作形成 happens-before 关系，我们肯定知道 X 的加载操作在第一个 store 之后，但在随后一个 store 之前，正如在图 3-2 所见。然而，它是否在第二个存储之前或之后观察值是不可预测的。换句话说，它可能是 1 或 2，但不是 0 或 3。 | |||
| 由于 join 和 spawn 操作形成 happens-before 关系，我们肯定知道 X 的 load 操作在第一个 store 之后，但在最后一个 store 之前，正如在图 3-2 所见。然而，它是否在第二个 store 操作之前或之后观察值是不可预测的。换句话说，它可能是 1 或 2，但不是 0 或 3。 | |||
There was a problem hiding this comment.
@hycinth22 这里需要一个讨论，关于 store 和 load 这两个原子操作该不该译。
因为 store 和 load 操作和方法它们共用名称，所以就没有像 compare-and-exchange 操作和 compare_exchange 方法那样区分，全部将 compare-and-exchange -> 「比较并交换」，或者将 fetch-and-motify -> 「获取并修改」。
如果粗略一点，可以不译 store、load。或者你对这两个原子操作有什么建议？
There was a problem hiding this comment.
我的意见的话，如果是明确指代代码中的特定操作，我比较倾向于作为众所周知的专有名词不译（就像没有人会翻译DNA这种词）。
这是基于这样的理由：首先代码肯定还是英文的、含有load/store等词，并且本书的读者也是需要理解这些代码的，所以我觉得我们可以合理认为，本书读者（或者至少读过此书介绍后）头脑中应当是有这些词的概念的。
所以，这种词翻译的意义可能并不大，并且即使我们翻译了，读者阅读时也还是需要想想英文是什么才能在示例代码中找到对应操作。
fwqaaq
reviewed
Jul 10, 2023
| @@ -138,10 +138,10 @@ fn f() { | |||
| } | |||
| ``` | |||
|
|
|||
| 由于 join 和产生操作形成 happens-before 关系，我们肯定知道 X 的加载操作在第一个 store 之后，但在随后一个 store 之前，正如在图 3-2 所见。然而，它是否在第二个存储之前或之后观察值是不可预测的。换句话说，它可能是 1 或 2，但不是 0 或 3。 | |||
| 由于 join 和 spawn 操作形成 happens-before 关系，我们肯定知道 X 的 load 操作在第一个 store 之后，但在最后一个 store 之前，正如在图 3-2 所见。然而，它是否在第二个 store 操作之前或之后观察值是不可预测的。换句话说，它可能是 1 或 2，但不是 0 或 3。 | |||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.