Skip to content

Raise events when connecting or disconnecting from a workspace for semantic changed#26512

Merged
TIHan merged 1 commit intodotnet:masterfrom
TIHan:RaiseChanged-WorkspaceReg
Jun 11, 2018
Merged

Raise events when connecting or disconnecting from a workspace for semantic changed#26512
TIHan merged 1 commit intodotnet:masterfrom
TIHan:RaiseChanged-WorkspaceReg

Conversation

@TIHan
Copy link
Copy Markdown
Contributor

@TIHan TIHan commented Apr 30, 2018

Connecting/disconnecting from a workspace was not kicking off semantic changed event. Talking with @jasonmalinowski and @Pilchie on Friday, it seems like it was an oversight and should be fixed.

@TIHan TIHan requested a review from a team as a code owner April 30, 2018 18:25
@TIHan
Copy link
Copy Markdown
Contributor Author

TIHan commented May 1, 2018

test windows_debug_spanish_unit32_prtest please

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

What's the end user impact of this change? In other words, how is this observable? Thanks!

@TIHan
Copy link
Copy Markdown
Contributor Author

TIHan commented May 1, 2018

On F#, we currently have a bug of semantic highlighting taking 3-8 seconds when you load a project; part of the reason is that the text buffer wasn't registered until after it tried produce tags from the semantic changed events. With the PR, a semantic change event kicks off when the a text buffer gets registered. This PR doesn't fix the F# issue entirely, only part of it.

C# does actually share a similar issue, its just not as noticeable due to semantic highlighting working before design time builds. This is not the case for F# as we have to wait for a designtime build.

@CyrusNajmabadi
Copy link
Copy Markdown
Contributor

Cool, thanks for the explanation!

@jinujoseph jinujoseph added this to the 15.8 milestone May 2, 2018
@TIHan
Copy link
Copy Markdown
Contributor Author

TIHan commented May 3, 2018

I can't merge this in myself. Can someone else do it?

@TIHan TIHan merged commit 48e1c7d into dotnet:master Jun 11, 2018
@TIHan TIHan deleted the RaiseChanged-WorkspaceReg branch June 11, 2018 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants