Информация об изменениях

Сообщение Re[6]: А кто у нас по T4 спец? от 03.04.2016 18:59

Изменено 03.04.2016 19:11 Sinix

Здравствуйте, IT, Вы писали:

IT>Похоже мы друг друга не поняли.

Ага, в n-й раз налезаю на всё те же грабли: коммит кода без объяснений Дальше буду issues гитхаба заводить, хотя бы чиста для истории.

UPD: коммент к коммиту


IT>Кстати, про using static CodeJam.DebugCode не понял. Оно там всё равно нигде не используется.

AndrewVK выше предложил.

Чтоб в других классах отладочных ассертов писать
[Conditional(DebugCondition), DebuggerHidden, MethodImpl(PlatformDependent.AggressiveInlining)]
а не
[Conditional(DebugCode.DebugCondition), DebuggerHidden, MethodImpl(PlatformDependent.AggressiveInlining)]

Заодно сейчас в оригинальном классе для PlatformDependent тоже using static сделаю. Аж забавно, как легко можно упустить очевидные моменты
Re[6]: А кто у нас по T4 спец?
Здравствуйте, IT, Вы писали:

IT>Похоже мы друг друга не поняли.

Ага, в n-й раз налезаю на всё те же грабли: коммит кода без объяснений Дальше буду issues гитхаба заводить, хотя бы чиста для истории.

UPD: Коммент к коммиту
UPD2: А можно в ReplaceInfo прикрутить bool FailIfNotFound? А то одна возможность накосячить всё-таки осталась неприкрытой

IT>Кстати, про using static CodeJam.DebugCode не понял. Оно там всё равно нигде не используется.

AndrewVK выше предложил.

Чтоб в других классах отладочных ассертов писать
[Conditional(DebugCondition), DebuggerHidden, MethodImpl(PlatformDependent.AggressiveInlining)]
а не
[Conditional(DebugCode.DebugCondition), DebuggerHidden, MethodImpl(PlatformDependent.AggressiveInlining)]

Заодно сейчас в оригинальном классе для PlatformDependent тоже using static сделаю. Аж забавно, как легко можно упустить очевидные моменты