Skip to content

Add tests and improve binary XmlDictionaryReader performance #73332

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
StephenMolloy merged 18 commits into from
Aug 15, 2022
Merged

Add tests and improve binary XmlDictionaryReader performance #73332

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
472e327
Read primitive types more efficient
Daniel-Svensson Jun 30, 2022
64ee4ba
Allow stream to read more than minimum number of required bytes
Daniel-Svensson Jul 5, 2022
b002d7c
Remove pinning and pointer code from array reads
Daniel-Svensson Jul 7, 2022
3b6ece8
Remove bounds checks from remaing array reads
Daniel-Svensson Jul 7, 2022
3b8e2e8
remove extra unchecked
Daniel-Svensson Jul 26, 2022
53d26a7
cleanup code
Daniel-Svensson Jul 26, 2022
068da74
merge upstream/main
Daniel-Svensson Jul 26, 2022
110346e
Start adding tests
Daniel-Svensson Jul 26, 2022
62467b7
Add tests for binary XmlDictionaryReader
Daniel-Svensson Jul 26, 2022
fad0457
cleanup
Daniel-Svensson Jul 27, 2022
d5ce3a7
revert back to old byte read
Daniel-Svensson Jul 31, 2022
5aee35c
add test for arrays using "ref" enumeration
Daniel-Svensson Aug 3, 2022
929ca23
add guid test
Daniel-Svensson Aug 3, 2022
c689762
Try to read while arrays from stream before processing
Daniel-Svensson Aug 3, 2022
bfea922
Write guid arrays as memory on LittleEndian platforms
Daniel-Svensson Aug 3, 2022
98b53cc
Merge remote-tracking branch 'upstream/main' into binaryxmlreader
Daniel-Svensson Aug 14, 2022
51b4957
Update tests to be independent of byte order
Daniel-Svensson Aug 14, 2022
9a2f8bd
FIx bugs introduced in #71752
Daniel-Svensson Aug 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
FIx bugs introduced in #71752 
* Add missing Advance to prevent corruption of reader
* Correctly read decimal values
  • Loading branch information
@Daniel-Svensson
Daniel-Svensson committed Aug 14, 2022
commit 9a2f8bd7f645fdf913402f51d3df96429ba7208f
18 changes: 10 additions & 8 deletions src/libraries/System.Private.DataContractSerialization/src/System/Xml/XmlBufferReader.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -390,11 +390,13 @@ public decimal ReadDecimal()
ReadOnlySpan<byte> bytes = buffer.AsSpan(offset, sizeof(decimal));
ReadOnlySpan<int> span = stackalloc int[4]
{
BinaryPrimitives.ReadInt32LittleEndian(bytes),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(8)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(12))
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(8, 4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(12, 4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(4, 4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(0, 4))
};

Advance(ValueHandleLength.Decimal);
return new decimal(span);
}
}
Expand Down Expand Up @@ -978,10 +980,10 @@ public decimal GetDecimal(int offset)
ReadOnlySpan<byte> bytes = _buffer.AsSpan(offset, sizeof(decimal));
ReadOnlySpan<int> span = stackalloc int[4]
{
BinaryPrimitives.ReadInt32LittleEndian(bytes),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(8)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(12))
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(8, 4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(12, 4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(4, 4)),
BinaryPrimitives.ReadInt32LittleEndian(bytes.Slice(0, 4))
};

return new decimal(span);
Expand Down