Skip to content

Commit 6468de2

Browse files
committed
Dispense with the lambda on _ShowLoadWarningsDialog
1 parent ea6a0d6 commit 6468de2

File tree

2 files changed

+5
-10
lines changed

2 files changed

+5
-10
lines changed

src/cascadia/TerminalApp/TerminalWindow.cpp

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -220,12 +220,7 @@ namespace winrt::TerminalApp::implementation
220220
_root->Initialized({ get_weak(), &TerminalWindow::_pageInitialized });
221221
_root->WindowSizeChanged({ get_weak(), &TerminalWindow::_WindowSizeChanged });
222222
_root->RenameWindowRequested({ get_weak(), &TerminalWindow::_RenameWindowRequested });
223-
_root->ShowLoadWarningsDialog([weakThis{ get_weak() }](auto&& /*s*/, const Windows::Foundation::Collections::IVectorView<Microsoft::Terminal::Settings::Model::SettingsLoadWarnings>& warnings) {
224-
if (auto strongThis{ weakThis.get() })
225-
{
226-
strongThis->_ShowLoadWarningsDialog(warnings);
227-
}
228-
});
223+
_root->ShowLoadWarningsDialog({ get_weak(), &TerminalWindow::_ShowLoadWarningsDialog });
229224
_root->Create();
230225

231226
AppLogic::Current()->SettingsChanged({ get_weak(), &TerminalWindow::UpdateSettingsHandler });
@@ -484,7 +479,7 @@ namespace winrt::TerminalApp::implementation
484479
// validating the settings.
485480
// - Only one dialog can be visible at a time. If another dialog is visible
486481
// when this is called, nothing happens. See ShowDialog for details
487-
void TerminalWindow::_ShowLoadWarningsDialog(const Windows::Foundation::Collections::IVectorView<SettingsLoadWarnings>& warnings)
482+
void TerminalWindow::_ShowLoadWarningsDialog(const IInspectable&, const Windows::Foundation::Collections::IVectorView<SettingsLoadWarnings>& warnings)
488483
{
489484
auto title = RS_(L"SettingsValidateErrorTitle");
490485
auto buttonText = RS_(L"Ok");
@@ -545,7 +540,7 @@ namespace winrt::TerminalApp::implementation
545540
}
546541
else if (settingsLoadedResult == S_FALSE)
547542
{
548-
_ShowLoadWarningsDialog(_initialLoadResult.Warnings());
543+
_ShowLoadWarningsDialog(nullptr, _initialLoadResult.Warnings());
549544
}
550545
}
551546

@@ -831,7 +826,7 @@ namespace winrt::TerminalApp::implementation
831826
}
832827
else if (args.Result() == S_FALSE)
833828
{
834-
_ShowLoadWarningsDialog(args.Warnings());
829+
_ShowLoadWarningsDialog(nullptr, args.Warnings());
835830
}
836831
else if (args.Result() == S_OK)
837832
{

src/cascadia/TerminalApp/TerminalWindow.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ namespace winrt::TerminalApp::implementation
189189
const winrt::hstring& contentKey,
190190
HRESULT settingsLoadedResult,
191191
const winrt::hstring& exceptionText);
192-
void _ShowLoadWarningsDialog(const Windows::Foundation::Collections::IVectorView<Microsoft::Terminal::Settings::Model::SettingsLoadWarnings>& warnings);
192+
void _ShowLoadWarningsDialog(const IInspectable& sender, const Windows::Foundation::Collections::IVectorView<Microsoft::Terminal::Settings::Model::SettingsLoadWarnings>& warnings);
193193

194194
bool _IsKeyboardServiceEnabled();
195195

0 commit comments

Comments
 (0)