-
Notifications
You must be signed in to change notification settings - Fork 4k
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.
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
Reduce allocations from newline information allocated by ChangedText.GetLinesCore #74728
Reduce allocations from newline information allocated by ChangedText.GetLinesCore #74728
Commits on Aug 13, 2024
-
Reduce allocations from newline information allocated by ChangedText.…
…GetLinesCore. Going to start this in draft mode and fire off a test insertion to verify this improves allocations as the typing scenario in speedometer scrolling test shows this as nearly 10% of allocations. The general idea here is that ChangedText doesn't need to keep track of line information as the SourceText that it wraps has that information. The complexity that was in ChangedText around newline splitting now needs to sit in both CompositeText and SubText as they need to understand how to expose their line collections.
Configuration menu - View commit details
-
Copy full SHA for de1d463 - Browse repository at this point
Copy the full SHA de1d463View commit details -
Configuration menu - View commit details
-
Copy full SHA for d3495be - Browse repository at this point
Copy the full SHA d3495beView commit details -
1) Modify tests to instead just take in a single string, and permute …
…over all possible source texts generatable from that sequence 2) Move the two added TextLineCollection derivations to be private.
Configuration menu - View commit details
-
Copy full SHA for 975340c - Browse repository at this point
Copy the full SHA 975340cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6ae36fc - Browse repository at this point
Copy the full SHA 6ae36fcView commit details -
Configuration menu - View commit details
-
Copy full SHA for 75741f2 - Browse repository at this point
Copy the full SHA 75741f2View commit details -
Configuration menu - View commit details
-
Copy full SHA for 03543db - Browse repository at this point
Copy the full SHA 03543dbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1c0bae9 - Browse repository at this point
Copy the full SHA 1c0bae9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 100549a - Browse repository at this point
Copy the full SHA 100549aView commit details
Commits on Aug 14, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 41397b8 - Browse repository at this point
Copy the full SHA 41397b8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7feb451 - Browse repository at this point
Copy the full SHA 7feb451View commit details
Commits on Aug 15, 2024
-
Configuration menu - View commit details
-
Copy full SHA for a402198 - Browse repository at this point
Copy the full SHA a402198View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0522300 - Browse repository at this point
Copy the full SHA 0522300View commit details
Commits on Aug 16, 2024
-
Simplification in CompositeText's handling of split \r\n. Instead of …
…handling in two different locations, just handle it in a single location up front.
Configuration menu - View commit details
-
Copy full SHA for 2dae6b9 - Browse repository at this point
Copy the full SHA 2dae6b9View commit details
Commits on Aug 19, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 97b4038 - Browse repository at this point
Copy the full SHA 97b4038View commit details
Commits on Aug 21, 2024
-
Update a comment and remove a condition that was preventing setting s…
…ome data in the empty SubText.UnderlyingSpan case.
Configuration menu - View commit details
-
Copy full SHA for 30f47ee - Browse repository at this point
Copy the full SHA 30f47eeView commit details