1
0
mirror of https://github.com/oliverbooth/X10D synced 2024-11-23 00:38:47 +00:00

fix: fix incorrect float/double being written for netstandard2.1

The call to _To_Bits yields the result containing the same bytes, but not the same value. This value was then stored as-is into the parameter, which causes a conversion on how the value is stored, ultimately causing the wrong value to be written to the stream.
This commit is contained in:
Oliver Booth 2023-04-01 17:11:05 +01:00
parent 677259b91c
commit 2815c505c3
No known key found for this signature in database
GPG Key ID: 20BEB9DC87961025
3 changed files with 106 additions and 40 deletions

View File

@ -145,6 +145,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
using the [GetOffsetAndLength](https://learn.microsoft.com/en-us/dotnet/api/system.range.getoffsetandlength?view=net-7.0) using the [GetOffsetAndLength](https://learn.microsoft.com/en-us/dotnet/api/system.range.getoffsetandlength?view=net-7.0)
method. method.
- X10D: `Stream.ReadSingle(Endianness)` now returns a float instead of a double. - X10D: `Stream.ReadSingle(Endianness)` now returns a float instead of a double.
- X10D: Fixed `Stream.ReadDouble(Endianness)`, `Stream.ReadSingle(Endianness)`, `Stream.WriteDouble(double, Endianness)`,
`Stream.WriteSingle(float, Endianness)` writing and reading the wrong endianness.
### Changed ### Changed

View File

@ -234,47 +234,99 @@ public class StreamTests
} }
[TestMethod] [TestMethod]
public void ReadDouble_WriteDouble_ShouldBeSymmetric() public void WriteDouble_ShouldWriteBigEndian_GivenBigEndian()
{ {
using var stream = new MemoryStream(); using var stream = new MemoryStream();
stream.Write(420.0, BitConverter.IsLittleEndian ? Endianness.LittleEndian : Endianness.BigEndian);
stream.Position = 0;
Assert.AreEqual(420.0, stream.ReadDouble(), 1e-6);
stream.Position = 0;
stream.Write(420.0, Endianness.LittleEndian);
stream.Position = 0;
Assert.AreEqual(420.0, stream.ReadDouble(Endianness.LittleEndian), 1e-6);
stream.Position = 0;
stream.Write(420.0, Endianness.BigEndian); stream.Write(420.0, Endianness.BigEndian);
Assert.AreEqual(8, stream.Position);
stream.Position = 0; stream.Position = 0;
Assert.AreEqual(420.0, stream.ReadDouble(Endianness.BigEndian), 1e-6);
Span<byte> actual = stackalloc byte[8];
ReadOnlySpan<byte> expected = stackalloc byte[] {0x40, 0x7A, 0x40, 0x00, 0x00, 0x00, 0x00, 0x00};
int read = stream.Read(actual);
byte[] actualArray = actual.ToArray();
byte[] expectedArray = expected.ToArray();
string actualBytes = string.Join(", ", actualArray.Select(b => $"0x{b:X2}"));
string expectedBytes = string.Join(", ", expectedArray.Select(b => $"0x{b:X2}"));
Trace.WriteLine($"Actual bytes: {actualBytes}");
Trace.WriteLine($"Expected bytes: {expectedBytes}");
Assert.AreEqual(8, read);
CollectionAssert.AreEqual(expectedArray, actualArray);
} }
[TestMethod] [TestMethod]
public void ReadDecimal_WriteSingle_ShouldBeSymmetric() public void WriteDouble_ShouldWriteLittleEndian_GivenLittleEndian()
{ {
using var stream = new MemoryStream(); using var stream = new MemoryStream();
stream.Write(420.0m, BitConverter.IsLittleEndian ? Endianness.LittleEndian : Endianness.BigEndian); stream.Write(420.0, Endianness.LittleEndian);
Assert.AreEqual(8, stream.Position);
stream.Position = 0; stream.Position = 0;
Assert.AreEqual(420.0m, stream.ReadDecimal());
stream.Position = 0; Span<byte> actual = stackalloc byte[8];
stream.Write(420.0m, Endianness.LittleEndian); ReadOnlySpan<byte> expected = stackalloc byte[] {0x00, 0x00, 0x00, 0x00, 0x00, 0x40, 0x7A, 0x40};
int read = stream.Read(actual);
stream.Position = 0; byte[] actualArray = actual.ToArray();
Assert.AreEqual(420.0m, stream.ReadDecimal(Endianness.LittleEndian)); byte[] expectedArray = expected.ToArray();
stream.Position = 0; string actualBytes = string.Join(", ", actualArray.Select(b => $"0x{b:X2}"));
stream.Write(420.0m, Endianness.BigEndian); string expectedBytes = string.Join(", ", expectedArray.Select(b => $"0x{b:X2}"));
Trace.WriteLine($"Actual bytes: {actualBytes}");
Trace.WriteLine($"Expected bytes: {expectedBytes}");
Assert.AreEqual(8, read);
CollectionAssert.AreEqual(expectedArray, actualArray);
}
[TestMethod]
public void WriteSingle_ShouldWriteBigEndian_GivenBigEndian()
{
using var stream = new MemoryStream();
stream.Write(420.0f, Endianness.BigEndian);
Assert.AreEqual(4, stream.Position);
stream.Position = 0; stream.Position = 0;
Assert.AreEqual(420.0m, stream.ReadDecimal(Endianness.BigEndian));
Span<byte> actual = stackalloc byte[4];
ReadOnlySpan<byte> expected = stackalloc byte[] {0x43, 0xD2, 0x00, 0x00};
int read = stream.Read(actual);
byte[] actualArray = actual.ToArray();
byte[] expectedArray = expected.ToArray();
string actualBytes = string.Join(", ", actualArray.Select(b => $"0x{b:X2}"));
string expectedBytes = string.Join(", ", expectedArray.Select(b => $"0x{b:X2}"));
Trace.WriteLine($"Actual bytes: {actualBytes}");
Trace.WriteLine($"Expected bytes: {expectedBytes}");
Assert.AreEqual(4, read);
CollectionAssert.AreEqual(expectedArray, actualArray);
}
[TestMethod]
public void WriteSingle_ShouldWriteLittleEndian_GivenLittleEndian()
{
using var stream = new MemoryStream();
stream.Write(420.0f, Endianness.LittleEndian);
Assert.AreEqual(4, stream.Position);
stream.Position = 0;
Span<byte> actual = stackalloc byte[4];
ReadOnlySpan<byte> expected = stackalloc byte[] {0x00, 0x00, 0xD2, 0x43};
int read = stream.Read(actual);
byte[] actualArray = actual.ToArray();
byte[] expectedArray = expected.ToArray();
string actualBytes = string.Join(", ", actualArray.Select(b => $"0x{b:X2}"));
string expectedBytes = string.Join(", ", expectedArray.Select(b => $"0x{b:X2}"));
Trace.WriteLine($"Actual bytes: {actualBytes}");
Trace.WriteLine($"Expected bytes: {expectedBytes}");
Assert.AreEqual(4, read);
CollectionAssert.AreEqual(expectedArray, actualArray);
} }
[TestMethod] [TestMethod]

View File

@ -996,27 +996,33 @@ public static class StreamExtensions
if (endianness == Endianness.LittleEndian) if (endianness == Endianness.LittleEndian)
{ {
#if NET5_0_OR_GREATER #if NET5_0_OR_GREATER
BinaryPrimitives.WriteSingleLittleEndian(buffer, value); BinaryPrimitives.WriteDoubleLittleEndian(buffer, value);
#else #else
if (BitConverter.IsLittleEndian) if (BitConverter.IsLittleEndian)
{ {
value = BinaryPrimitives.ReverseEndianness(BitConverter.SingleToInt32Bits(value)); MemoryMarshal.Write(buffer, ref value);
}
else
{
int temp = BinaryPrimitives.ReverseEndianness(BitConverter.SingleToInt32Bits(value));
MemoryMarshal.Write(buffer, ref temp);
} }
System.Runtime.InteropServices.MemoryMarshal.Write(buffer, ref value);
#endif #endif
} }
else else
{ {
#if NET5_0_OR_GREATER #if NET5_0_OR_GREATER
BinaryPrimitives.WriteSingleBigEndian(buffer, value); BinaryPrimitives.WriteDoubleBigEndian(buffer, value);
#else #else
if (BitConverter.IsLittleEndian) if (BitConverter.IsLittleEndian)
{ {
value = BinaryPrimitives.ReverseEndianness(BitConverter.SingleToInt32Bits(value)); int temp = BinaryPrimitives.ReverseEndianness(BitConverter.SingleToInt32Bits(value));
MemoryMarshal.Write(buffer, ref temp);
}
else
{
MemoryMarshal.Write(buffer, ref value);
} }
System.Runtime.InteropServices.MemoryMarshal.Write(buffer, ref value);
#endif #endif
} }
@ -1064,10 +1070,13 @@ public static class StreamExtensions
#else #else
if (BitConverter.IsLittleEndian) if (BitConverter.IsLittleEndian)
{ {
value = BinaryPrimitives.ReverseEndianness(BitConverter.DoubleToInt64Bits(value)); MemoryMarshal.Write(buffer, ref value);
}
else
{
long temp = BinaryPrimitives.ReverseEndianness(BitConverter.DoubleToInt64Bits(value));
MemoryMarshal.Write(buffer, ref temp);
} }
System.Runtime.InteropServices.MemoryMarshal.Write(buffer, ref value);
#endif #endif
} }
else else
@ -1077,10 +1086,13 @@ public static class StreamExtensions
#else #else
if (BitConverter.IsLittleEndian) if (BitConverter.IsLittleEndian)
{ {
value = BinaryPrimitives.ReverseEndianness(BitConverter.DoubleToInt64Bits(value)); long temp = BinaryPrimitives.ReverseEndianness(BitConverter.DoubleToInt64Bits(value));
MemoryMarshal.Write(buffer, ref temp);
}
else
{
MemoryMarshal.Write(buffer, ref value);
} }
System.Runtime.InteropServices.MemoryMarshal.Write(buffer, ref value);
#endif #endif
} }