RC1
От: AndrewVK Россия http://blogs.rsdn.org/avk
Дата: 20.04.16 20:45
Оценка:
Сабж.
https://www.nuget.org/packages/CodeJam/1.0.0-rc1
С этого момента все багфиксы и мелкие доработки, которые должны попасть в релиз, нужно мержить в ветку 1.0.0-release. Функционал на следующий релиз остается в master.
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
AVK Blog
Re: RC1
От: Sinix  
Дата: 21.04.16 05:45
Оценка:
Здравствуйте, AndrewVK, Вы писали:

AVK>Сабж.

AVK>https://www.nuget.org/packages/CodeJam/1.0.0-rc1
AVK>С этого момента все багфиксы и мелкие доработки, которые должны попасть в релиз, нужно мержить в ветку 1.0.0-release. Функционал на следующий релиз остается в master.

Если дело к релизу — убери плиз вот эту перегрузку в Code.
    public static void AssertArgument(
            bool condition,
            string argName,
            Func<string> messageFactory)


Она исходную проблему не решает. Ну да, строка не форматируется, но аллокации на каждый вызов остаются (из-за замыканий).
Для таких случаев у нас есть
if (failedAssertion) throw CodeExceptions.Argument(...);


Важно, т.к. ассерты по коду расставляются тысячами, любой косяк в дизайне будет заиспользован. Добавлять в часто вызываемый метод ассерт, который замусоривает GC0 — точно не самая лучшая идея
Re[2]: RC1
От: AndrewVK Россия http://blogs.rsdn.org/avk
Дата: 21.04.16 10:26
Оценка: 2 (1)
Здравствуйте, Sinix, Вы писали:

S>Если дело к релизу — убери плиз вот эту перегрузку в Code.


Убрал
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
AVK Blog
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.