6.11. Custom Allocators #28
Merged
Conversation
This file contains hidden or 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
dalance
force-pushed
the
custom-allocators
branch
from
2837e39 to
7ca878c
Compare
| <!-- allocate and deallocate memory. The standard library is not compiled assuming --> | ||
| <!-- either one, and the compiler will decide which allocator is in use at --> | ||
| <!-- compile-time depending on the type of output artifact being produced. --> | ||
| 現在コンパイラは `alloc_system` と `alloc_jemalloc` (いくつかのターゲットにはありません)という2つの既定のアロケータを提供しています。これらのアロケータは単に普通のRustのクレートで、メモリの割り当てと解放の手続きを実装しています。標準ライブラリはどちらか一方を前提としてコンパイルされているわけではありません。コンパイラは生成する成果物の種類に応じてどちらのアロケータを使用するかをコンパイル時に決定します。 |
There was a problem hiding this comment.
[nits]
いくつかのターゲットにはありません
ここのsomeは「いくつかの」の意味ではなくそういうものもある、の意味と思います。なので
「jemallocがないターゲットもあります」はどうでしょう。
|
コメントしました。[WIP]を外したタイミングでpingを飛ばして頂けるとレスポンスが早くなります。 コードブロックのコメントの原文についてはここで考えた方が良さそうですね。何か良いアイディアありますか？ |
|
コードブロックのコメントですが、以下のように行頭に'# `を付けると（少なくとも見た目上は）コメントアウトできるような気がしますが、 書いてから気づきましたが、'# 'はRunするときに有効になるのですね。 |
|
ご指摘いただいた部分の修正と、漏れていた部分の追加を行いました。 |
良さそうですね。ソースコード中のコメントはこの方式にしましょう。 |
|
対訳表がコンフリクトしたのでそこだけ修正お願いします。 |
Closed
dalance
force-pushed
the
custom-allocators
branch
from
714f969 to
198d739
Compare
|
対訳表を修正しました。 |
|
ありがとうございます。マージします。 |
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.
ひとまず翻訳完了しました。
ソースコード中のコメントは削除するとのことでしたが、結構分量がありましたので
確認の際に比較しやすいかと思い、とりあえず残してあります。
訳に問題がなくなった時点で削除しようと思います。
よろしくお願いします。