docs: improve disclaimer regarding LinkedList#65574
Conversation
|
(rust_highfive has picked a reviewer for you, use r? to override) |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
|
r? @Centril |
|
r=me rollup when green. |
|
(Also, note for the future: It's best to avoid @-pinging folks since it tends to generate more notifications when rebasing etc.) |
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
a458a4a to
7c08bea
Compare
|
The job Click to expand the log.I'm a bot! I can only do what humans tell me to, so if this was not helpful or you have suggestions for improvements, please ping or otherwise contact |
7c08bea to
9fdc913
Compare
|
r? @Centril |
There was a problem hiding this comment.
Why the ... at the end now?
There was a problem hiding this comment.
A period (.) feels abrupt. The ... indicates a gentle connection between the before (claim) and after (justification).
Also, am sure there's a better explanation why people use ..., which is something of a subtle concept.
There was a problem hiding this comment.
I often use "..." in casual conversation because it suggests that I'm not done thinking; it's a sort of "uhm", "eh". but in a more formal setting (i.e. standards documentation) it just feels sloppy. See also https://en.wikipedia.org/wiki/Speech_disfluency and https://en.wikipedia.org/wiki/Filler_(linguistics).
There was a problem hiding this comment.
Alternative:
NOTE: It is almost always better to use
VecorVecDequebecause array-based containers are generally faster...
As a sidenote, and assuming you like that better, do you think we should add a comma there...
NOTE: It is almost always better to use
VecorVecDeque, because array-based containers are generally faster...
There was a problem hiding this comment.
The first alternative is fine.
9fdc913 to
a844e59
Compare
|
@bors r+ rollup |
|
📌 Commit a844e59 has been approved by |
…Centril docs: improve disclaimer regarding LinkedList
Rollup of 10 pull requests Successful merges: - #65136 (Update codegen option documentation.) - #65574 (docs: improve disclaimer regarding LinkedList) - #65720 (Add FFI bindings for LLVM's Module::getInstructionCount()) - #65905 ([doc] fixes for unix/vxworks `OpenOptionsExt::mode`) - #65962 (Fix logic in example.) - #66019 (Improved std::iter::Chain documentation) - #66038 (doc(str): show example of chars().count() under len()) - #66042 (Suggest correct code when encountering an incorrect trait bound referencing the current trait) - #66073 (Do not needlessly write-lock) - #66096 (Add a failing UI test for multiple loops of all kinds in a `const`) Failed merges: r? @ghost
No description provided.