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

Сообщение Re[51]: MS забило на дотнет. Питону - да, сишарпу - нет? от 10.09.2021 18:39

Изменено 10.09.2021 18:40 vdimas

Старый текст
Новый текст
Разница

Re[51]: MS забило на дотнет. Питону - да, сишарпу - нет?

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

Вариант на FindChildV, который немного подшаманил.

https://sharplab.io/#v2:EYLgxg9gTgpgtADwGwBYA0AXEBDAzgWwB8ABAJgEYBYAKGIAYACY8gOgBEBLbAcwDsJcGDmFwBuGvSasAcgFd8MKMLETGzFgCVZvIQpYBhCPgAOHADaKAyooBuwmCtprWWnRz0BJHYojHrUOzAHcScpTW1dGBYvDCVeXGUQyXVXSOidOISRFgANAA4kJMZLAAtsKGMAGWxgcLcFEJoAeiaAbQApDgwAcRheRWEACgwAT2MYCAAzQYAhbTASgB4ZiARFyphsSekIABMYRY4dAD40BiOMY+OASmuAXRpWy1jZMAxqkYhZDEGPr4wANJHXYsawAR1kfSE2DM9xoF0UvBhDEEUFeGAYc14C0WABVjjQAN40Bik87eKBIswMXEMAD6u2wGGwdBCZPJGERyNpDKZ2HIIQAxMYoDx8NgGAB3cq8I7cBi7Di4GoWBj6Sx0VAAThJZIRlO59MZzNIbL1FKpNKNfIAzGbSfrLTzjdgUPaOVzqc6+QBWd2Ow285mFXUOi2Bl0Adn94a91uZeRjnINcaD2C1Sc9VrTAEFMynsy6ZvmnfHsPoSxG+WxK6mXQBRWuFvkAMSFIrFEullLlDFggmgMDVGu1oYYLWINr7MEmVowJSVrQu5P2CDuAF5jrBZwBZGD4aAjHflXBlMwGWBMmCWYzYXiDbdluhnchIa5L3iru4hAC+NHhsZSEgTCkJi8wlPWCAYsS1Dsh0XS9P0ShgMMYwTNMFy3A8sFkpOQHTrOtIeJy+B4scD4zgw85KmB2JLPiTDENRuBnMuwIwAg1wMJuBEMAAqvEWxRDmuy7BRs7EExC64CwabPiuHHXO6eHMMBj6XrsEC8GYIxWhomy7AA8tpIzEfuZGDEcVHSbROIMZJzGsToCmcdxxy8QJyqTMJonifxgneSwOa4Pp0wOdJskunQ1xOauSk0H+1DNG0nQ9H0AwoaM4xTL8mzbHsBwXDcZxZehlk6FhjypYhGWodl0wzKKdE7Ps6x5S1hUnE5lwxVRaE5ZhcLUAG1KouimJNQsHWLNIZwMSADAeJUSoYB1HgmGYyyTSU02zTSpz7WO7KSiUihDtIDALWNbxnEtK1rRtM1zQS1AweyIocDYV62UsRX0sC9iOO9ShfZyP3LKsT37e5dILOYuywLw7ofaDQ7LrDC5mAjfSGBEjQ4aSeEaVpOlztJH4Yuxa5jm97Jkq0e7zns63GGYgyMyUzMbYZxhCFpMk5tw3D9gkNgwF4ZhHHKQ102S3AwBgR1krTsvsmwMDALI3BBbguCKD8VMMMc64MIwABkZsuQwixql8FXuqrDBfVADDDKxXEmy2wL6JjuwAGrlXFDuq8QkZUSwHVLt+Sukol7KJWORMGSTunLrjzk8Rj8OI+nGLuYwAD8/2foDmgGcZOlmfggxZ1jOd2xicAMOQXELay/4Ex6BbEyZDD6dgRkmTed4Q2se34u5PvZ30bkMHuB5QEeJ5nhemycv3g86cP96Pp5Qk66FNdw3XfSRXy0VnLX2O8Ln8VJZ3I3TgPKd9+XQ+3rwhwnItJdBLgs/z0PMeKAp4YSryvBvCuIxt5+T3oFYKh86QAz/mfZkF96TH2vrffG7JH491JpA9+I8/YwDeNAUgPokBf0uO5EhZCoBeEVH/AB+4gHLzAfoPAGBqFnDoRgchlDqFXHZqwxewDQHnn0Jedeb8t4f1gQFYSIUZw12QQ4VBLJeqvluPjMcDMFac12CzNmHMuasx5nzeIQUhYi0+uLbSUteDcBlrhKc+DdKzEhuPdymDuouS4l7T8U8saWHKpTT8ikaYxyduUFcgNuI/yYcEaJztzgJPbp3dkkxoCu1EGkm2aiZIbCcfOBgFs4l/yjtbE2VM8kcAANT1K4sHQmYdBjBOvlHViRtTYMCLobJuhSlwMCbuQO4l1/HugTp3fRTMjEbREXM4xFiOD82scLBwot7GS1lE4lxhM3HJ17p4sez0GC+I5P4hggTdgdIDmxCJnEomZLJCcqG+JupcX0hgWQlJyoTTvFNAq7z3J6zMJMPxmCdjXGieyHigwwWTAML7RGrQoUQDuLFDiIzXbop6YXFEMBwXpCSbgNFvsdg4rGRM6Kd8O6O1SWABJfCBFIHAZyQOikWkxJdhKE2GTHbZMvAsV2qT8DkgYCyhhv8HAwpeaSFWjseVUQSTmGwCBSAGCMLeWA3RpGKFxGUe8+AzhgDvkqskqSICqvVZqncEAxbHlwAAa2GDrSwcoLCDFuAwQgpsEAtkDUGugAqLXKuMAkmYXQeaKCZKsqxuJRTmDlAALR8LnQYgxZCYQgEpccTQGBfBduKF15x/55DgMALoUoOD7G5eyRlvsEkSnqQwYw9a9SzkGJgo2Jsr71wiHKsNdNQ6vx+X8xyGCUU4wblS81YaOBdojTbPIQ6w2KuHaO75vz7yTswfOi1ccw0tpNomaJR7ZZboVju4Y0lL6YIHc5UZB6GDTLgqY+ZrNFmGOWbzeNAsbGbLsRLRxzjsLsjwm87x5zfZ+KpgE72vswn+LHJuaJhsbZIJlTJNMYdCB+v7TOiI1sGA6nlXTfpjySNYdJRoqc+Gp3TxvrOm2PpYWq0o6uajhSNHkF9QRh9RHnI2xtOxi1nHsWYZ43JMTw6+mu0I7wDR8l4OyeHQtI+06lNpnIFihAOKaOlzkmuuTdMFoYeLrRtMoEGOKdziRlAamw1F000xjRpA9MGeky6FuTmLUacU3RzzTdDMoOsyZuT5mqNSewxon0/HGMn2Y8Rm2kY/MUatjFqzLoUAJbsyxhgSB0uOxc4FtM6ArYhe87aCLpnZYBcE9pl0PpguWaMzl2rdWotcay+1vkwFbONfsyu4rHGFONY0UgVroX1Fph9J1urZIGtaY0ZGab1XgwLbM5ltrYWix5aGwV8gonyMlZ2zNnDLotQHa08N5uVBTvOfOxt7AeQbtMbu+QOgo3ZalYm2mPI63Yu4a24t5bbm0xaiB9lvkq6fsTIsxdjROZ3tJc+6QeH7I/srdzNDvrzItSg66+NnHRY8d7b5DmIn9XnvA5dGwVHWCjtsce+J2nMPmT6EZ4+jENtyCOdZ3J7HEOXT6HJ7Nos1PIsk5F9WcXl2+T6Cl47brkndsS75PWbnQnefNyK4L4dwuksaPrPLjRbBldhvB8btMLYzdpnrHS6giVH76kmNgIIi1lqCA6jMPABx8REkTlOWkzEKYuXGYSBg8sMB5IvUwKcadZ1R5j3HhKAFkyWjdx7ocd0ffAvml7+6BU/d6zIkH53HdXcUnd57vPq0CrGJBRM+vvv/fl87idM6DALpXReDdIv+f9hN+8RXl3gEyCYi8WchareC8vWOqdWAPeJnXQwLdb3Dfh+PVH69YPU+EDeoYFHt9riV8dV0Z3PCSfiM8Q6lqvG9Kz+h/Jg8r8s9I5U2jpX++j818MA2C2GmkLzn231ZjaiAPn2egr2Bk+m+ixDsnchdGRhBm+nRkgEf07hRm+m3jIgYGCm3hYQXiXhARXikTXmvHkUfBdGU3vQbhfUfmIFyy9jMDZlpC+jMEhEhXoOeUdnVk1m1mCj1igB+AwOExNm0W5VhlnRNjENj2iUATEXYUkX1RgWoL5FoPOXoJYBYLZg4MhCd3jifwOTJkXDfw4nA2VmiRj1nhoKrk5U4m5T1gxB4jsJIgcI9hiU4JgCmX3xvwzhhjkN/CAA=

Где только возможно стоит использовать readonly-модификатор метода, ReadOnlySpan.
Одно плохо, кривая сигнатура MemoryMarshal.CreateReadOnlySpan:

        [MethodImpl(MethodImplOptions.AggressiveInlining)]
        public static ReadOnlySpan<T> CreateReadOnlySpan<T>(ref T reference, int length) => new ReadOnlySpan<T>(ref reference, length);

Должно было быть так:

        [MethodImpl(MethodImplOptions.AggressiveInlining)]
        public static ReadOnlySpan<T> CreateReadOnlySpan<T>(in T reference, int length) => new ReadOnlySpan<T>(Unsafe.AsRef(reference), length);

Потому что в readonly-методах приходится делать так:

internal readonly ReadOnlySpan<Box<N, T>> Children => MemoryMarshal.CreateReadOnlySpan(ref Unsafe.AsRef(_children._data0), _childrenCount);

Что легко лечится своим каким-нить хелпером
public static SpanHelper {
[MethodImpl(MethodImplOptions.AggressiveInlining)]
public static ReadOnlySpan<T> CreateReadOnlySpan<T>(in T reference, int length) => MemoryMarshal.CreateReadOnlySpan(Unsafe.AsRef(reference), length);

}

Re[51]: MS забило на дотнет. Питону - да, сишарпу - нет?

        [MethodImpl(MethodImplOptions.AggressiveInlining)]
        public static ReadOnlySpan<T> CreateReadOnlySpan<T>(ref T reference, int length) => new ReadOnlySpan<T>(ref reference, length);

Должно было быть так:

        [MethodImpl(MethodImplOptions.AggressiveInlining)]
        public static ReadOnlySpan<T> CreateReadOnlySpan<T>(in T reference, int length) => new ReadOnlySpan<T>(Unsafe.AsRef(reference), length);

Потому что в readonly-методах приходится делать так:

internal readonly ReadOnlySpan<Box<N, T>> Children => MemoryMarshal.CreateReadOnlySpan(ref Unsafe.AsRef(_children._data0), _childrenCount);

Что лечится своим каким-нить хелпером

public static SpanHelper {
    [MethodImpl(MethodImplOptions.AggressiveInlining)]
    public static ReadOnlySpan<T> CreateReadOnlySpan<T>(in T reference, int length) => MemoryMarshal.CreateReadOnlySpan(Unsafe.AsRef(reference), length);
}