mirror of
https://github.com/microsoft/terminal.git
synced 2026-02-04 05:35:20 +00:00
these are reasonable nits
This commit is contained in:
@@ -1,3 +1,4 @@
|
||||
|
||||
// Copyright (c) Microsoft Corporation.
|
||||
// Licensed under the MIT license.
|
||||
|
||||
@@ -2430,10 +2431,7 @@ namespace winrt::TerminalApp::implementation
|
||||
TermControl term{ settings.DefaultSettings(), settings.UnfocusedSettings(), connection };
|
||||
|
||||
// GH#12515: ConPTY assumes it's hidden at the start. If we're not, let it know now.
|
||||
if (_visible)
|
||||
{
|
||||
term.WindowVisibilityChanged(_visible);
|
||||
}
|
||||
term.WindowVisibilityChanged(_visible);
|
||||
|
||||
if (_hostingHwnd.has_value())
|
||||
{
|
||||
|
||||
@@ -312,10 +312,7 @@ namespace winrt::Microsoft::Terminal::TerminalConnection::implementation
|
||||
THROW_IF_FAILED(_CreatePseudoConsoleAndPipes(dimensions, flags, &_inPipe, &_outPipe, &_hPC));
|
||||
|
||||
// GH#12515: The conpty assumes it's hidden at the start. If we're visible, let it know now.
|
||||
if (_initialVisibility)
|
||||
{
|
||||
THROW_IF_FAILED(ConptyShowHidePseudoConsole(_hPC.get(), _initialVisibility));
|
||||
}
|
||||
THROW_IF_FAILED(ConptyShowHidePseudoConsole(_hPC.get(), _initialVisibility));
|
||||
if (_initialParentHwnd != 0)
|
||||
{
|
||||
THROW_IF_FAILED(ConptyReparentPseudoConsole(_hPC.get(), reinterpret_cast<HWND>(_initialParentHwnd)));
|
||||
|
||||
@@ -487,13 +487,8 @@ extern "C" HRESULT WINAPI ConptyClearPseudoConsole(_In_ HPCON hPC)
|
||||
// - For more information, refer to GH#12515.
|
||||
extern "C" HRESULT WINAPI ConptyShowHidePseudoConsole(_In_ HPCON hPC, bool show)
|
||||
{
|
||||
const PseudoConsole* const pPty = (PseudoConsole*)hPC;
|
||||
HRESULT hr = pPty == nullptr ? E_INVALIDARG : S_OK;
|
||||
if (SUCCEEDED(hr))
|
||||
{
|
||||
hr = _ShowHidePseudoConsole(pPty, show);
|
||||
}
|
||||
return hr;
|
||||
// _ShowHidePseudoConsole will return E_INVALIDARG for us if the hPC is nullptr.
|
||||
return _ShowHidePseudoConsole((PseudoConsole*)hPC, show);
|
||||
}
|
||||
|
||||
// - Sends a message to the pseudoconsole informing it that it should use the
|
||||
|
||||
Reference in New Issue
Block a user