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 — точно не самая лучшая идея
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.