[master] Update dependencies from aspnet/Extensions#1234
Merged
dotnet-maestro[bot] merged 1 commit intomasterfrom Oct 13, 2019
Merged
[master] Update dependencies from aspnet/Extensions#1234dotnet-maestro[bot] merged 1 commit intomasterfrom
dotnet-maestro[bot] merged 1 commit intomasterfrom
Conversation
…0191012.1 - Microsoft.AspNetCore.BenchmarkRunner.Sources - 5.0.0-alpha1.19512.1 - Microsoft.AspNetCore.Testing - 5.0.0-alpha1.19512.1 - Microsoft.Extensions.CommandLineUtils.Sources - 5.0.0-alpha1.19512.1 - Microsoft.Extensions.HashCodeCombiner.Sources - 5.0.0-alpha1.19512.1 - Microsoft.Extensions.NonCapturingTimer.Sources - 5.0.0-alpha1.19512.1 - Microsoft.Extensions.Logging - 5.0.0-alpha1.19512.1
Contributor
Author
Auto-Merge StatusThis pull request has been merged because the following merge policies have succeeded.
|
dotnet-maestro
bot
deleted the
darc-master-06db6701-1701-41d9-813e-5238c1ee42ed
branch
DustinCampbell
added a commit
that referenced
this pull request
Sep 8, 2025
> [!TIP] > I strongly recommend reviewing this commit-by-commit. The commit history represents a progression of changes, and each commit has a detailed message describing its purpose. This change overhauls the HTML character entity reference decoding performed by `ComponentMarkupEncodingPass`. The Razor compiler makes a best effort to decode HTML content for code generation. If it determines that the content is suitable as raw text and all character entity references can be decoded (if any), the compiler's code gen will call `RenderTreeBuilder.AddContent(...)`. If not, it'll call `RenderTreeBuilder.AddMarkupContent(...)`, which can have an impact on the runtime performance of the compiled application. So, the goal is to call `AddContent(...)` as much as possible. While tuning the performance of `ComponentMarkupEncodingPass` to reduce allocations and avoid unnecessary work, I encountered two bugs: 1. The Razor compiler doesn't implement hexadecimal HTML character entity references correctly. Hex character entity references are supposed to be in the form, `&#xhhhh;`. However, the compiler's implementation only supports an illegal form, `�xhhhh;` and doesn't actually support the legal form. So, if content contains legal hex numeric character entity references, they will fail to decode and `RenderTreeBuilder.AddMarkupContent(...)` will be code-gen'd. 2. When decoding a decimal character entity reference, such as `Ӓ`, a lookup table is used to retrieve the code point value. However, the lookup table isn't complete and doesn't include very simple values. For example, it doesn't include `/` which translates to a `/` character. So, if the content contains decimal numeric character entity references that aren't included in the lookup table, they will fail to decode and `RenderTreeBuilder.AddMarkupContent(...)` will be code-gen'd. I've gone ahead and fixed both of these issues in 6f23ef0. In addition, there are several performance and allocation fixes in this commit. These changes resulted in the creation of a handful of useful helpers for building and creating strings using a `MemoryBuilder<ReadOnlyMemory<char>>`. Those helpers and tests are included in e266160. Many thanks to Copilot for help with writing all of the tests. 🤖🧪❤️ ---- CI Build: https://dev.azure.com/dnceng/internal/_build/results?buildId=2786521&view=results Test Insertion: https://dev.azure.com/devdiv/DevDiv/_git/VS/pullrequest/667364 Toolset Run: https://dev.azure.com/dnceng/internal/_build/results?buildId=2786524&view=results
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the following dependencies
From https://github.com/aspnet/Extensions