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.

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

Investigate use of Editor Snapshots to improve performance #15

Open
mrlacey opened this issue Dec 2, 2018 · 5 comments
Open

Investigate use of Editor Snapshots to improve performance #15

mrlacey opened this issue Dec 2, 2018 · 5 comments
Milestone

Comments

@mrlacey
Copy link
Owner

mrlacey commented Dec 2, 2018

Look to improve performance when updating the UI through the use of snapshots.
See https://docs.microsoft.com/en-us/visualstudio/extensibility/inside-the-editor?view=vs-2017#a-closer-look-at-the-text-model-and-the-text-view

Also various entries in https://github.com/Microsoft/VSSDK-Extensibility-Samples/

@mrlacey mrlacey added this to the Backlog milestone Dec 2, 2018
@mrlacey
Copy link
Owner Author

mrlacey commented Dec 22, 2018

Also, investigate using e.NewOrReformattedLines in ResourceAdornmentManager.LayoutChangedHandler
rather than iterating through all lines every time.

@mrlacey mrlacey self-assigned this Jun 2, 2020
@MagicAndre1981
Copy link

As the extension fails to load in VS2017 (see #39 ), I tried it in VS2019 and here I get perf warnings

string_vis_extension_18s_delay

is this related to this issue?

@mrlacey
Copy link
Owner Author

mrlacey commented Aug 30, 2021

As the extension fails to load in VS2017 (see #39 ), I tried it in VS2019 and here I get perf warnings

string_vis_extension_18s_delay

is this related to this issue?

It's possibly related. I haven't been able to track down the cause of the perf warning.

@mrlacey
Copy link
Owner Author

mrlacey commented Oct 11, 2021

Sorry for the delayed response. Recent illness has stopped me from working on anything but I hope to get to this eventually.

@MagicAndre1981
Copy link

Sorry for the delayed response. Recent illness has stopped me from working on anything but I hope to get to this eventually.

ok, hope you get well soon

@mrlacey mrlacey removed their assignment Jan 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants