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

vova lantsov

Но делать 2 перегрузки такое себе. наверное никак атрибут применить...

Можешь курнуть мануал

23:56пожаловаться #1

Там в аттрмбут битовая маска передается

23:56пожаловаться #2

Которая отвечает за дженерики

23:56пожаловаться #3

vova lantsov in pro.net

Ilya Chernoudov

Там в аттрмбут битовая маска передается

Ты про AttributeTargets?

23:58пожаловаться #4

2020 January 08

vova lantsov

Ты про AttributeTargets?

Я про NotNull

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

vova lantsov in pro.net

У NotNull нет перегрузок и его нельзя применить к generic'у, либо я не понял о чём ты

00:02пожаловаться #6

мне кажется, вы о разных NotNull говорите

00:06пожаловаться #7

есть же C#8 и решарперовский

00:06пожаловаться #8

Oleg Morozov in pro.net

на дженерик параметр ретурн тайпа нельзя вешать аттрибуты

00:11пожаловаться #9

Oleg Morozov in pro.net

физически

00:11пожаловаться #10

потому в C#8 фиче это обошли
https://sharplab.io/#v2:D4YgdgrgNlCGBGUCmACJYHILACgACATCgOqwAuuA3rirXgMwp4CMAbE8wKwB0AarFAhI8rADwsADAH4AfCgAqSAM5kAFAEoatajlp6UZABYAnAPYB3FJBgBuLSgC+uJziA==

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

[return: System.Runtime.CompilerServices.Nullable(new byte[] {
0,
2
})]
public static ValueTask<string> Test()
{
throw null;
}

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

Oleg Morozov in pro.net

не понял причем здесь нуллейблы, если речь про аттрибуты для дженерик параметра

00:17пожаловаться #13

vova lantsov in pro.net

Я про атрибуты из System.Diagnostics.CodeAnalysis

00:18пожаловаться #14

JIT: Optimize "constant_string".Length by EgorBo · Pull Request #1378 · dotnet/runtime

попытка номер 2 https://github.com/dotnet/runtime/pull/1378

GitHub

Resurrects dotnet/coreclr#26000
Example (showcases why it's better to have this opt in JIT instead of Roslyn)
bool Validate(string str)
{
return str.Length > 0 &&...

00:38пожаловаться #15

т.к. ироды закрыли мой ПР в корцлр репе в связи с миграцией

00:38пожаловаться #16

Вот еще этот кейс пофикшу https://sharplab.io/#v2:EYLgxg9gTgpgtADwGwBYA0AXEBDAzgWwB8ABAJgEYBYAKGPKQAIyGBhBgbxpoZ6foeAQIAGwYBJXADkArsOEAKAMoYoASwB2Acwa4VASm69O1XqaYB2HSoYBeGw3WzhAbkM8AvlxO86jQSIYAFRhdeQNvHmMzH0sJGTl5ACIACRg5CES9VwiGT2p3IA=

01:03пожаловаться #17

не удаляются сравнения константных строк с наллом

01:03пожаловаться #18

это очень бесит когда вы делаете так:

"fuck you".AsSpan()

01:04пожаловаться #19

то там внутри не выпиливается проверка на налл