Skip to content
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

Jump to bottom

Send over the new sourcetext's content checksum when sending notification of textchanges #75928

Merged
merged 3 commits into from
Nov 16, 2024
Merged

Send over the new sourcetext's content checksum when sending notification of textchanges #75928

merged 3 commits into from
Nov 16, 2024
+22 −16

Commits on Nov 15, 2024

  1. Send over the new sourcetext's content checksum when sending notifica… 

    …tion of textchanges

Calculation of this on the server was taking quite a bit of CPU (about 11.6% in the scrolling speedometer during the typing scenario). Instead, pass this data over as part of the text change notification, similar to what SerializableSourceText's serialization does.
    @ToddGrun
    ToddGrun committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    1fe4c49 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'dotnet:main' into dev/toddgrun/SendOverSourceTextTextCh… 

    …ecksumInSynchronizationNotification
    @ToddGrun
    ToddGrun authored Nov 15, 2024
    Configuration menu
    Copy the full SHA
    4dd93dc View commit details
    Browse the repository at this point in the history

  3. Erroneously was using Checksum.From when converting from a content ha… 

    …sh when should be using Checksum.Create
    @ToddGrun
    ToddGrun committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    35e9e71 View commit details
    Browse the repository at this point in the history