Skip to content
This repository has been archived by the owner on Mar 7, 2018. It is now read-only.

Добавление абзаца #67

Merged
merged 2 commits into from
Feb 20, 2017
Merged

Добавление абзаца #67

merged 2 commits into from
Feb 20, 2017

Conversation

zanziver
Copy link
Contributor

@zanziver zanziver commented Feb 20, 2017

Добавление абзаца с описанием использования звездочки ( * ) на ссылках.


This change is Reviewable

Добавление абзаца с описанием использования звездочки на ссылках
@@ -131,6 +131,9 @@ println!("{}", x);
также должно быть помечено как `mut`. Если бы этого не было, то мы не могли бы
получить изменяемую ссылку неизменяемого значения.

Вы также заметили что мы добавили звездочку (`*`) перед `y`, сделано это для того,
чтобы получить доступ непосредственно к содержимому на которое указывает ссылка.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

На мой взгляд лучше так.

Вы также заметили что мы добавили звездочку (`*`) перед `y`. Это сделано из-за того, 
что `y` - это изменяемая ссылка. Вам нужно использовать (`*`), для того чтобы получить
доступ непосредственно к содержимому на которое указывает ссылка.

Хотя возможно есть вариант получше, чем " Это сделано из-за того, что y - это изменяемая ссылка."

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот как раз это предложение меня и смутило, тем более что в предыдущем абзаце уже об этом в общем-то упоминается. Но так ближе к оригиналу конечно, а лучше формулировка что-то в голову не приходит.
Только предлагаю тогда акцентировать внимание на том что ссылка сама данных не содержит. Мне кажется так предложение будет логически завершенным.

Вы также заметили что мы добавили звездочку (`*`) перед `y`. Это сделано из-за того, 
что `y` - это ссылка и сама данных не содержит. Вам нужно использовать (`*`), для того
 чтобы получить доступ непосредственно к содержимому на которое указывает ссылка.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Согласен. Присылай изменения и можно мержить.

@vessd vessd merged commit a73ede4 into rust-lang-ru:master Feb 20, 2017
@vessd
Copy link
Member

vessd commented Feb 20, 2017

Спасибо!

@zanziver zanziver deleted the add_paragraph branch February 20, 2017 15:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants