-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Дока на пример правильной обработки цикла ожидания завершения процесса #402 #406
Дока на пример правильной обработки цикла ожидания завершения процесса #402 #406
Conversation
/cc @EvilBeaver @dmpas |
https://msdn.microsoft.com/en-us/library/f8hahtxf.aspx Можно обернуть пример в тэги code и example. Сам не пробовал, но в MSDN есть такое. |
Да, могу поправить. |
да, СП в 1с-синтакс обрабатывает тег example https://github.com/xDrivenDevelopment/vsc-language-1c-bsl/blob/master/src/features/oscriptStdLib.ts#L259 - выводит его отдельным абзацем с заголовком Пример |
@dmpas Выделил тег примеров. |
Думаю, в примере стоит пояснить, в чём "правильность". Или отсылку дать куда-нибудь. А то так со стороны не понять, мне кажется. А в остальном всё хорошо. |
@dmpas Я упорный :) |
У меня больше нет вопросов. @EvilBeaver ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Копипаст мюиз гиттера это хорошо, но не совсем точно. Речь не только про stderr, а вообще про потоки вывода. Смысл в том что читать надо оба. Просьба переформулировать
Андрей, а самому сложно потратить минуту и напрямую в моем PR поправить 2 строчки, не? |
Мое упорство также имеет пределы :( |
@artbear ты сам меня приучил докапываться до мелочей, разве нет? Там суть как раз в том, что вообще про потоки речь, а не про только stderr. Я думал ты взялся это описать, а выяснилось, что получился копипаст с гиттера, в котором формулировки не совсем понятные. В результате из этого текста можно понять, что обязательно читать нужно только stderr, а stdout и сам закроется. А это не так. |
/// Процесс не завершается, пока любоой из потоков (stdout, stderr) открыт для чтения. /// Процесс висит и ждет, пока его освободят от текста в обоих потоках.
В честь праздника сам на гитхабе исправил.
@EvilBeaver Устроит? PS поехали на Таганку пива пить? |
На таганку могу в будни только |
/// Процесс не завершается, пока поток stderr открыт для чтения | ||
/// Он висит в процессах и ждет, пока его освободят от текста в stderr | ||
/// Процесс не завершается, пока любоой из потоков (stdout, stderr) открыт для чтения. | ||
/// Процесс висит и ждет, пока его освободят от текста в обоих потоках. | ||
/// | ||
/// Пока НЕ Процесс.Завершен ИЛИ Процесс.ПотокВывода.ЕстьДанные ИЛИ Процесс.ПотокОшибок.ЕстьДанные Цикл | ||
/// Если ПериодОпросаВМиллисекундах <> 0 Тогда |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Надо исправить на Если ПериодОпросаВМиллисекундах <>
0 Тогда
потому что <!-- Проигнорирован некорректный комментарий XML для члена "P:ScriptEngine.HostedScript.Library.ProcessContext.HasExited" -->
Дока на пример правильной обработки цикла ожидания завершения процесса #402