~ruther/NosSmooth

b3dfda624d82463ae0cf9faf1fdde1da6ee60ed1 — František Boháček 2 years ago 3ff55e7
fix(packets): correctly add packets with multiple headers
1 files changed, 8 insertions(+), 5 deletions(-)

M Packets/NosSmooth.PacketSerializer/Packets/PacketTypesRepository.cs
M Packets/NosSmooth.PacketSerializer/Packets/PacketTypesRepository.cs => Packets/NosSmooth.PacketSerializer/Packets/PacketTypesRepository.cs +8 -5
@@ 43,6 43,14 @@ public class PacketTypesRepository : IPacketTypesRepository
    /// <returns>A result that may or may not have succeeded.</returns>
    public Result AddPacketType(Type type)
    {
        if (type.FullName is not null)
        {
            if (_typeToPacket.ContainsKey(type.FullName))
            { // The packet was already added.
                return Result.FromSuccess();
            }
        }

        if (!typeof(IPacket).IsAssignableFrom(type))
        {
            return new ArgumentInvalidError


@@ 93,11 101,6 @@ public class PacketTypesRepository : IPacketTypesRepository
        var info = new PacketInfo(header.Identifier, type, converter);
        if (type.FullName is not null)
        {
            if (_typeToPacket.ContainsKey(type.FullName))
            { // The packet was already added.
                return Result.FromSuccess();
            }

            _typeToPacket[type.FullName] = info;
        }


Do not follow this link