diff options
| author | Laurențiu Nicola <lnicola@dend.ro> | 2020-10-01 21:02:39 +0300 |
|---|---|---|
| committer | Laurențiu Nicola <lnicola@dend.ro> | 2020-10-01 21:02:39 +0300 |
| commit | 223374969205059f3448ef9e9d7ab1c44d5fa8cf (patch) | |
| tree | a9fe9f827708ec3164d52083d15dfc59b77f229d /docs/dev | |
| parent | 3f4e9914ff77c76ad3cbdebe1e4e2c0a78818d63 (diff) | |
| download | rust-223374969205059f3448ef9e9d7ab1c44d5fa8cf.tar.gz rust-223374969205059f3448ef9e9d7ab1c44d5fa8cf.zip | |
Improve grammar and fix code example in style guide
Diffstat (limited to 'docs/dev')
| -rw-r--r-- | docs/dev/style.md | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/docs/dev/style.md b/docs/dev/style.md index bcd86fd3f07..f0fdb5adc44 100644 --- a/docs/dev/style.md +++ b/docs/dev/style.md @@ -197,7 +197,7 @@ fn frobnicate(walrus: Option<Walrus>) { } ``` -Avoid preconditions that spawn function boundaries: +Avoid preconditions that span across function boundaries: ```rust @@ -218,9 +218,8 @@ fn foo() { } // Not as good -fn is_string_literal(s: &str) -> Option<&str> { +fn is_string_literal(s: &str) -> bool { s.starts_with('"') && s.ends_with('"') - Some() } fn foo() { @@ -231,8 +230,8 @@ fn foo() { } ``` -In the "Not as good" version, the precondition that `1` is a valid char boundary is checked in `is_string_literal` and utilized in `foo`. -In the "Good" version, precondition check and usage are checked in the same block, and then encoded in the types. +In the "Not as good" version, the precondition that `1` is a valid char boundary is checked in `is_string_literal` and used in `foo`. +In the "Good" version, the precondition check and usage are checked in the same block, and then encoded in the types. # Early Returns |
