Skip to content

Commit abe1143

Browse files
committed
Fixup OmniSharp fallback AnalyzerConfigOption updater
1 parent 9cb6907 commit abe1143

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

src/Features/ExternalAccess/OmniSharp/Options/OmniSharpSolutionAnalyzerConfigOptionsUpdater.cs

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -40,14 +40,14 @@ internal static bool UpdateOptions(Workspace workspace, OmniSharpEditorConfigOpt
4040

4141
// add o# option values:
4242
var lineFormattingOptions = editorConfigOptions.LineFormattingOptions;
43-
AddOption(FormattingOptions2.UseTabs, lineFormattingOptions.UseTabs, builder);
44-
AddOption(FormattingOptions2.TabSize, lineFormattingOptions.TabSize, builder);
45-
AddOption(FormattingOptions2.IndentationSize, lineFormattingOptions.IndentationSize, builder);
46-
AddOption(FormattingOptions2.NewLine, lineFormattingOptions.NewLine, builder);
43+
AddOption(FormattingOptions2.UseTabs, lineFormattingOptions.UseTabs);
44+
AddOption(FormattingOptions2.TabSize, lineFormattingOptions.TabSize);
45+
AddOption(FormattingOptions2.IndentationSize, lineFormattingOptions.IndentationSize);
46+
AddOption(FormattingOptions2.NewLine, lineFormattingOptions.NewLine);
4747

4848
var implementTypeOptions = editorConfigOptions.ImplementTypeOptions;
49-
AddOption(ImplementTypeOptionsStorage.InsertionBehavior, (ImplementTypeInsertionBehavior)implementTypeOptions.InsertionBehavior, builder);
50-
AddOption(ImplementTypeOptionsStorage.PropertyGenerationBehavior, (ImplementTypePropertyGenerationBehavior)implementTypeOptions.PropertyGenerationBehavior, builder);
49+
AddOption(ImplementTypeOptionsStorage.InsertionBehavior, (ImplementTypeInsertionBehavior)implementTypeOptions.InsertionBehavior);
50+
AddOption(ImplementTypeOptionsStorage.PropertyGenerationBehavior, (ImplementTypePropertyGenerationBehavior)implementTypeOptions.PropertyGenerationBehavior);
5151

5252
var newFallbackOptions = oldFallbackOptions.SetItem(
5353
LanguageNames.CSharp,
@@ -61,14 +61,13 @@ internal static bool UpdateOptions(Workspace workspace, OmniSharpEditorConfigOpt
6161
throw ExceptionUtilities.Unreachable();
6262
}
6363

64-
static void AddOption<T>(
64+
void AddOption<T>(
6565
PerLanguageOption2<T> option,
66-
T value,
67-
ImmutableDictionary<string, string>.Builder builder)
66+
T value)
6867
{
6968
var configName = option.Definition.ConfigName;
7069
var configValue = option.Definition.Serializer.Serialize(value);
71-
builder.Add(configName, configValue);
70+
builder[configName] = configValue;
7271
}
7372
}
7473
}

0 commit comments

Comments
 (0)