SergVasiliev Dec 27 2021 at 13:15

Ошибки и подозрительные места в исходниках .NET 6

19 min

13K

PVS-Studio corporate blog.NET*C#*Open source*

+29

Comments 4

csl Dec 27 2021 at 18:48

Последний коммит проверяемой ветки с GitHub (runtime) за 22-ое октября.

https://github.com/dotnet/runtime/tree/v6.0.0

8-го ноября выпустили "ускоренную версию" Apple Silicon в .NET 6.0 https://habr.com/ru/company/dododev/blog/593307/

Правильно ли я понимаю, что "ускоренная" (в посте выше указано, что до релиза от 8.11.2022 рантайм на ARM работал медленно) ARM-версия рантайма не тестировалась?

SergVasiliev Dec 28 2021 at 06:41

Я так понимаю, это одна и та же версия - в конце октября её зафиксировали, в ноябре зарелизили.

Однако я проверял только код C# библиотек, до самого рантайма не добрался. :(

LoadRunner Dec 27 2021 at 20:07

Фактически данный метод осуществляет маппинг с CompressionOption на CompressionLevel. Подозрительно здесь то, что значения CompressionOption.Normal и CompressionOption.Maximum маппятся на значение CompressionLevel.Optimal.

А то, что Fast и SuperFast маппятся на Fastest, анализатор не смутило?

SergVasiliev Dec 28 2021 at 08:39

Сейчас сделано так, что если два дубля, то предупреждение выдаётся на первый. Поэтому на второй кейс предупреждение не было выдано. Будет, если поправить описанный в статье дубль.

Если же говорить про ошибочность, то случай с Fastest выглядит менее подозрительным (с учётом элементов в CompressionLevel).