Телеграмм чат группы pro

мне отвечали

01:37пожаловаться #1

Grigorii Kuzmin

я просто репортил на красный код

01:37пожаловаться #2

Ɖrēw

багу репортишь прям из райдера

01:37пожаловаться #3

или из студии, если решарпер

01:37пожаловаться #4

Прикольно, не знал, что так можно

01:38пожаловаться #5

Ну я уже закинул кому надо

01:38пожаловаться #6

более того, это рекомендуют сами чуваки из жб)

01:38пожаловаться #7

т.к. чем больше разных кейсов, тем им проще фиксить

01:38пожаловаться #8

Volodymyr

лол какого хера даже простой

value is int?

боксит?

потому что должен, вопрос тут почему не оптимизирует

01:40пожаловаться #9

а так должен

01:40пожаловаться #10

это же генерик

01:40пожаловаться #11

Volodymyr in pro.net

EgorBo

это же генерик

Блин, точно

01:43пожаловаться #12

https://github.com/mono/mono/pull/18286

GitHub

[WIP] Optimize `box + isinst + unbox.any` by EgorBo · Pull Request #18286 · mono/mono

Optimize boxing in some cases with Nullable, e.g. simplified case:
public static int? Foo(T value)
{
return value as int?;
}

public static int? Foo2()
{
// Foo1 will be in...

02:31пожаловаться #13

сперва затащил в моно, но в корцлр фикс будет выглядить примерно так же

02:32пожаловаться #14

на уровне импорта ил

02:32пожаловаться #15

кейс судя по диффам популярный

02:32пожаловаться #16

https://github.com/mono/mono/pull/18286

EgorBo

GitHub

[WIP] Optimize `box + isinst + unbox.any` by EgorBo · Pull Request #18286 · mono/mono

Optimize boxing in some cases with Nullable, e.g. simplified case:
public static int? Foo(T value)
{
return value as int?;
}

public static int? Foo2()
{
// Foo1 will be in...

У тебя там Foo а не Foo1

02:33пожаловаться #17

В примере

02:33пожаловаться #18

фиксед

02:33пожаловаться #19

Roman Bukin in pro.net

Ayrat Hudaygulov

Ну смотреть только на версию не нужно. Есть проекты, которые де факто юзабельны до версии 1.0.0

Где то улыбается один Handbrake (13 лет в бете)