[clang-tidy] Remove in-product uses of std::bind

src/cascadia/TerminalControl/ControlCore.cpp

@@ -86,25 +86,25 @@ namespace winrt::Microsoft::Terminal::Control::implementation
  // GH#8969: pre-seed working directory to prevent potential races
  _terminal->SetWorkingDirectory(_settings->StartingDirectory());
 
- auto pfnCopyToClipboard = std::bind(&ControlCore::_terminalCopyToClipboard, this, std::placeholders::_1);
+ auto pfnCopyToClipboard = [this](auto&& PH1) { _terminalCopyToClipboard(std::forward<decltype(PH1)>(PH1)); };
  _terminal->SetCopyToClipboardCallback(pfnCopyToClipboard);
 
- auto pfnWarningBell = std::bind(&ControlCore::_terminalWarningBell, this);
+ auto pfnWarningBell = [this] { _terminalWarningBell(); };
  _terminal->SetWarningBellCallback(pfnWarningBell);
 
- auto pfnTitleChanged = std::bind(&ControlCore::_terminalTitleChanged, this, std::placeholders::_1);
+ auto pfnTitleChanged = [this](auto&& PH1) { _terminalTitleChanged(std::forward<decltype(PH1)>(PH1)); };
  _terminal->SetTitleChangedCallback(pfnTitleChanged);
 
- auto pfnScrollPositionChanged = std::bind(&ControlCore::_terminalScrollPositionChanged, this, std::placeholders::_1, std::placeholders::_2, std::placeholders::_3);
+ auto pfnScrollPositionChanged = [this](auto&& PH1, auto&& PH2, auto&& PH3) { _terminalScrollPositionChanged(std::forward<decltype(PH1)>(PH1), std::forward<decltype(PH2)>(PH2), std::forward<decltype(PH3)>(PH3)); };
  _terminal->SetScrollPositionChangedCallback(pfnScrollPositionChanged);
 
- auto pfnTerminalTaskbarProgressChanged = std::bind(&ControlCore::_terminalTaskbarProgressChanged, this);
+ auto pfnTerminalTaskbarProgressChanged = [this] { _terminalTaskbarProgressChanged(); };
  _terminal->TaskbarProgressChangedCallback(pfnTerminalTaskbarProgressChanged);
 
- auto pfnShowWindowChanged = std::bind(&ControlCore::_terminalShowWindowChanged, this, std::placeholders::_1);
+ auto pfnShowWindowChanged = [this](auto&& PH1) { _terminalShowWindowChanged(std::forward<decltype(PH1)>(PH1)); };
  _terminal->SetShowWindowCallback(pfnShowWindowChanged);
 
- auto pfnPlayMidiNote = std::bind(&ControlCore::_terminalPlayMidiNote, this, std::placeholders::_1, std::placeholders::_2, std::placeholders::_3);
+ auto pfnPlayMidiNote = [this](auto&& PH1, auto&& PH2, auto&& PH3) { _terminalPlayMidiNote(std::forward<decltype(PH1)>(PH1), std::forward<decltype(PH2)>(PH2), std::forward<decltype(PH3)>(PH3)); };
  _terminal->SetPlayMidiNoteCallback(pfnPlayMidiNote);
 
  auto pfnCompletionsChanged = [=](auto&& menuJson, auto&& replaceLength) { _terminalCompletionsChanged(menuJson, replaceLength); };

src/cascadia/WindowsTerminal/AppHost.cpp

@@ -73,10 +73,7 @@ AppHost::AppHost(const winrt::TerminalApp::AppLogic& logic,
  _window->SetMinimizeToNotificationAreaBehavior(_windowLogic.GetMinimizeToNotificationArea());
 
  // Tell the window to callback to us when it's about to handle a WM_CREATE
- auto pfn = std::bind(&AppHost::_HandleCreateWindow,
- this,
- std::placeholders::_1,
- std::placeholders::_2);
+ auto pfn = [this](auto&& PH1, auto&& PH2) { _HandleCreateWindow(std::forward<decltype(PH1)>(PH1), std::forward<decltype(PH2)>(PH2)); };
  _window->SetCreateCallback(pfn);
 
  _windowCallbacks.MouseScrolled = _window->MouseScrolled({ this, &AppHost::_WindowMouseWheeled });
@@ -394,10 +391,7 @@ void AppHost::Initialize()
  // while the screen is off.
  TerminalTrySetAutoCompleteAnimationsWhenOccluded(static_cast<::IUnknown*>(winrt::get_abi(_windowLogic.GetRoot())), true);
 
- _window->SetSnapDimensionCallback(std::bind(&winrt::TerminalApp::TerminalWindow::CalcSnappedDimension,
- _windowLogic,
- std::placeholders::_1,
- std::placeholders::_2));
+ _window->SetSnapDimensionCallback([this](auto&& PH1, auto&& PH2) { return _windowLogic.CalcSnappedDimension(std::forward<decltype(PH1)>(PH1), std::forward<decltype(PH2)>(PH2)); });
 
  // Create a throttled function for updating the window state, to match the
  // one requested by the pty. A 200ms delay was chosen because it's the

src/host/VtInputThread.cpp

@@ -42,11 +42,11 @@ VtInputThread::VtInputThread(_In_ wil::unique_hfile hPipe,
  _pInputStateMachine = std::make_unique<StateMachine>(std::move(engine));
 
  // we need this callback to be able to flush an unknown input sequence to the app
- auto flushCallback = std::bind(&StateMachine::FlushToTerminal, _pInputStateMachine.get());
+ auto flushCallback = [capture0 = _pInputStateMachine.get()] { return capture0->FlushToTerminal(); };
  engineRef->SetFlushToInputQueueCallback(flushCallback);
 
  // we need this callback to capture the reply if someone requests a status from the terminal
- _pfnSetLookingForDSR = std::bind(&InputStateMachineEngine::SetLookingForDSR, engineRef, std::placeholders::_1);
+ _pfnSetLookingForDSR = [engineRef](auto&& PH1) { engineRef->SetLookingForDSR(std::forward<decltype(PH1)>(PH1)); };
 }
 
 // Function Description:

src/host/srvinit.cpp

@@ -882,7 +882,7 @@ PWSTR TranslateConsoleTitle(_In_ PCWSTR pwszConsoleTitle, const BOOL fUnexpand,
 
  // Set up the renderer to be used to calculate the width of a glyph,
  // should we be unable to figure out its width another way.
- auto pfn = std::bind(&Renderer::IsGlyphWideByFont, static_cast<Renderer*>(g.pRender), std::placeholders::_1);
+ auto pfn = [ObjectPtr = static_cast<Renderer*>(g.pRender)](auto&& PH1) { return ObjectPtr->IsGlyphWideByFont(std::forward<decltype(PH1)>(PH1)); };
  SetGlyphWidthFallback(pfn);
  }
  catch (...)