Open DavidKarlas opened 5 years ago
Problem is that if https://github.com/mono/monodevelop/blob/master/main/src/addins/CSharpBinding/MonoDevelop.CSharp.Formatting/CSharpFormatter.cs#L99 goes to background thread and eventually needs to come back to UI thread it can't because https://github.com/mono/monodevelop/blob/master/main/src/addins/CSharpBinding/MonoDevelop.CSharp.Formatting/CSharpFormatter.cs#L73 is holding it. I can repro this consistently by trying to move some field/local declaration in and out of method by using Option+Up/Down.
Maybe we should remove the sync API.
Problem is that if https://github.com/mono/monodevelop/blob/master/main/src/addins/CSharpBinding/MonoDevelop.CSharp.Formatting/CSharpFormatter.cs#L99 goes to background thread and eventually needs to come back to UI thread it can't because https://github.com/mono/monodevelop/blob/master/main/src/addins/CSharpBinding/MonoDevelop.CSharp.Formatting/CSharpFormatter.cs#L73 is holding it. I can repro this consistently by trying to move some field/local declaration in and out of method by using Option+Up/Down.