Remove unnecessary FMT (#17795)

carlos-zamora · DHowett · commit 85afa41c32f3 · 2024-09-04T14:55:14.000-05:00
There's an unnecessary `fmt::format` here caught in the code review: #17678 (comment) This simply removes it. (cherry picked from commit 93d592b) Service-Card-Id: PVTI_lADOAF3p4s4AmhmQzgSwIS4 Service-Version: 1.22
diff --git a/src/cascadia/TerminalSettingsModel/Command.cpp b/src/cascadia/TerminalSettingsModel/Command.cpp
@@ -774,7 +774,7 @@ namespace winrt::Microsoft::Terminal::Settings::Model::implementation
                 // covers actions w/out args
                 // - "command": "unbound" --> "unbound"
                 // - "command": "copy"    --> "copy"
-                changes.emplace(fmt::format(FMT_COMPILE("{}"), json.asString()));
+                changes.emplace(json.asString());
             }
             else
             {

Original file line number	Diff line number	Diff line change
`@@ -774,7 +774,7 @@ namespace winrt::Microsoft::Terminal::Settings::Model::implementation`
`774`	`774`	`// covers actions w/out args`
`775`	`775`	`// - "command": "unbound" --> "unbound"`
`776`	`776`	`// - "command": "copy" --> "copy"`
`777`		`- changes.emplace(fmt::format(FMT_COMPILE("{}"), json.asString()));`
	`777`	`+ changes.emplace(json.asString());`
`778`	`778`	`}`
`779`	`779`	`else`
`780`	`780`	`{`