Conversation
| { | ||
| public int ID; | ||
| public string Name; | ||
| public string AlterName; |
There was a problem hiding this comment.
Везде в программе используем для англ. синонима термин Alias. Предлагаю сделать единообразно и в этом случае.
|
|
||
| argValues = unboxedArgs; | ||
|
|
||
| } while (false); |
There was a problem hiding this comment.
Не совсем понятна магия с do.. while(false). Для чего нужен однопроходный цикл?
И второе - не лучше ли вынесли вот этот type 2 subroutine в отдельный метод? Код вызова конструктора и так переусложнен, его неплохо бы порефакторить.
| var td = new TypeDescriptor() | ||
| { | ||
| Name = alias, | ||
| AlterName = alterAlias, |
There was a problem hiding this comment.
Поменять на термин "Alias" (см. выше)
|
@DJYar тесты попадали в большом количестве. Просьба исправить: http://build.oscript.io/job/1Script/view/change-requests/job/PR-721/5/testReport/ |
|
@EvilBeaver я PR прикрою, чтобы в CI он не собирался. Время свободное появится -- у себя правки внесу и открою другой PR с учетом замечаний. |
|
@DJYar конечно, это некропостинг, но может вернемся к вопросу? |
2 participants
Распаковка аргументов из массива (issue #700 )
Проверял на 8.3.11.2867 и в OneScript-- теперь код работает одинаково.