Skip to content
This repository has been archived by the owner on Jan 16, 2022. It is now read-only.

Перевод статьи из медиума за 6-е марта #92

Merged
merged 2 commits into from
Mar 10, 2015
Merged

Перевод статьи из медиума за 6-е марта #92

merged 2 commits into from
Mar 10, 2015

Conversation

alexpods
Copy link

@alexpods alexpods commented Mar 7, 2015

Добавить в crowdin не смог, наверное нет доступа.

UPD Добавил предоложения по переводам статей из медиума #94

@gko
Copy link
Contributor

gko commented Mar 7, 2015

я сделал таску и импортировал оригинал.

@alexpods
Copy link
Author

alexpods commented Mar 7, 2015

@nrvru Спасибо. Поправил опечатки. Не стал изменять только ["выпустил" => "вышел"], тк там указкно кто выпустил (@rvagg):

On Friday, March 6th, @rvagg released io.js v1.5.0. The complete change log can be found on GitHub.

Пока коммиты не сквошу, тк думаю будут еще правки.

@gko
Copy link
Contributor

gko commented Mar 7, 2015

@alexpods а где заголовок и первое предложение?
Может перенести перевод на сrowdin?


* **buffer**: Новый метод `Buffer#indexOf()`, созданный на подобии метода [`Array#indexOf()`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf). Может принимать в качестве аргумента строку, Buffer или число. Строки интерпретируются как UTF8. (Trevor Norris) [#561](https://github.com/iojs/io.js/pull/561)
* **fs**: Объект параметров `options` в методах модуля `'fs'` больше не выполняет проверку `hasOwnProperty()`, следовательно, позволяя объектам параметров иметь свойства в прототипах, которые будут применяться. (Jonathan Ong) [#635](https://github.com/iojs/io.js/pull/635)
* **tls**: Компания PayPal сообщила о вероятной утечке памяти в TLS. Некоторые недавние изменения в **stream_wrap** оказались виновными в этом. Первоночальная правка расположена в [#1078](https://github.com/iojs/io.js/pull/1078), вы можете следить за прогрессом устранения утечки в [#1075](https://github.com/iojs/io.js/issues/1075) (Fedor Indutny).
Copy link
Contributor

Choose a reason for hiding this comment

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

Некоторые недавние изменения в stream_wrap оказались виновными в этом. => Причина скорее всего лежит в недавних изменениях stream_wrap. Исправление можете найти по [ссылка], а за прогрессом устранения утечки следить тут.

Copy link
Author

Choose a reason for hiding this comment

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

Такой перевод немного отходит от оригинала:

Some of the recent changes in stream_wrap appear to be to blame.

"apper to be" тут переводится как "оказались", т.е. виновники уже найдены. "скорее всего" звучит как "мы не знаем точно в где проблема, но подозреваем что там".

Может быть перевести так: "Причина оказалась в недавних изменениях stream_wrap. Исправление можете найти по [ссылка], а за прогрессом устранения утечки следите тут."?

@alexpods
Copy link
Author

alexpods commented Mar 7, 2015

@gorodinskiy Я ориганал брал отсюда. Там нет ни заголовка ни первого предложения. Их добавили специально для медиума.

Сейчас уже я не вижу смысла добавить перевод в crowdin: статья переведена, как минимум три человека её просмотрело и внесли свои предложения. Но если кто-то хочет этим заняться - я не против.

Внес большинство предложенных правок. Несколько из них вызвали сомнения. @Olegas и @gorodinskiy посмотрите.

@artcygn
Copy link
Contributor

artcygn commented Mar 7, 2015

Может быть лучше так: Полный список изменений можно найти на GitHub-е.

@alexpods
Copy link
Author

alexpods commented Mar 9, 2015

@artcygn Да, так лучше. Поправил.


### Основные изменения

* **buffer**: Новый метод `Buffer#indexOf()`, созданный наподобие метода [`Array#indexOf()`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/indexOf). Может принимать в качестве аргумента строку, Buffer или число. Строки интерпретируются как UTF8. (Trevor Norris) [#561](https://github.com/iojs/io.js/pull/561)
Copy link
Contributor

Choose a reason for hiding this comment

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

наподобие -> "работающий как" или "подобный методу" ?

@YurySolovyov
Copy link
Contributor

1 комент

@gko
Copy link
Contributor

gko commented Mar 9, 2015

перечитал. апрув если что.

@alexpods
Copy link
Author

alexpods commented Mar 9, 2015

Добавил последние предложенные правки. Засквошил все коммиты с фиксами. Думаю завтра утром надо выкладывать на хабр.

@Olegas
Copy link
Contributor

Olegas commented Mar 9, 2015

Я выложу. Пока подготовлю черновик, завтра внесу какие-то правки, если будут, и опубликую утром

@Olegas
Copy link
Contributor

Olegas commented Mar 9, 2015

Черновик сделал. Если до 9 утра не будет замечаний - публикую.

@Olegas
Copy link
Contributor

Olegas commented Mar 10, 2015

@Olegas Olegas closed this Mar 10, 2015
@alexpods alexpods reopened this Mar 10, 2015
alexpods pushed a commit that referenced this pull request Mar 10, 2015
Перевод статьи из медиума за 6-е марта
@alexpods alexpods merged commit 5a260dd into nodejs:master Mar 10, 2015
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.

5 participants