MiloszKrajewski / K4os.Compression.LZ4

LZ4/LH4HC compression for .NET Standard 1.6/2.0 (formerly known as lz4net)
MIT License
675 stars 77 forks source link

Dependency to System.Memory is not needed for .NET Standard 2.1 and .NET 5.0 #63

Closed tibel closed 2 years ago

tibel commented 2 years ago

In newer frameworks (.NET Standard 2.1 and .NET 5.0) Span is supported out of the box and the NuGet package is not needed.

MiloszKrajewski commented 2 years ago

I'll take a look. I did not use framework dependencies with paket yet, you I will have to find out how first.

redwyre commented 2 years ago

This is something I am very interested in as well as the extra references cause issues with Unity

MiloszKrajewski commented 2 years ago

1.2.16 should do exactly this

redwyre commented 2 years ago

Thanks @MiloszKrajewski! Are you able to make the same change in https://github.com/MiloszKrajewski/K4os.Hash.xxHash ?

MiloszKrajewski commented 2 years ago

@redwyre xxHash 1.0.7 has same changes applied. Note LZ4 still references 1.0.6 so you need to manually reference xxHash 1.0.7. I'll change it in next release.