Improvements to TerminalInput #966

Closed
opened 2026-01-30 22:12:38 +00:00 by claunia · 2 comments
Owner

Originally created by @Nicholas-Baron on GitHub (May 10, 2019).

This line could use = default syntax. This would also reduce the number of lines in the associated cpp.

bf460ab7fe/src/terminal/input/terminalInput.hpp (L26)

Originally created by @Nicholas-Baron on GitHub (May 10, 2019). This line could use ` = default` syntax. This would also reduce the number of lines in the associated cpp. https://github.com/microsoft/Terminal/blob/bf460ab7feed7e203f1c5cf1bc06a1c732742870/src/terminal/input/terminalInput.hpp#L26
Author
Owner

@DHowett-MSFT commented on GitHub (May 11, 2019):

Hey Nicholas,
Thanks for the report. We’re glad to accept pull requests for issues like this!


From: Nicholas Baron notifications@github.com
Sent: Friday, May 10, 2019 3:30:01 PM
To: microsoft/Terminal
Cc: Subscribed
Subject: [microsoft/Terminal] Improvements to TerminalInput (#685)

This line could use = default syntax. This would also reduce the number of lines in the associated cpp.

bf460ab7fe/src/terminal/input/terminalInput.hpp (L26)https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FTerminal%2Fblob%2Fbf460ab7feed7e203f1c5cf1bc06a1c732742870%2Fsrc%2Fterminal%2Finput%2FterminalInput.hpp%23L26&data=01%7C01%7Cduhowett%40microsoft.com%7C04c35bbc97b9461602b708d6d5970abd%7C72f988bf86f141af91ab2d7cd011db47%7C1&sdata=H6ih1sI%2B7kBB9zz2oCmdVqR5ITumQ%2BfDrfu%2FA8CH1JY%3D&reserved=0


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FTerminal%2Fissues%2F685&data=01%7C01%7Cduhowett%40microsoft.com%7C04c35bbc97b9461602b708d6d5970abd%7C72f988bf86f141af91ab2d7cd011db47%7C1&sdata=r9ZC3yMUHZLAbhs8PJJ%2Bl1Ss2Py%2BB%2FZTA59TkHYU5V0%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADNHLGW5ROOZCAES27BZAPTPUXZOTANCNFSM4HMGTLTA&data=01%7C01%7Cduhowett%40microsoft.com%7C04c35bbc97b9461602b708d6d5970abd%7C72f988bf86f141af91ab2d7cd011db47%7C1&sdata=1zzpQbZuJbRdwSCtPUH66JRJDwLxqyQnl44cGA0%2BzpQ%3D&reserved=0.

@DHowett-MSFT commented on GitHub (May 11, 2019): Hey Nicholas, Thanks for the report. We’re glad to accept pull requests for issues like this! ________________________________ From: Nicholas Baron <notifications@github.com> Sent: Friday, May 10, 2019 3:30:01 PM To: microsoft/Terminal Cc: Subscribed Subject: [microsoft/Terminal] Improvements to TerminalInput (#685) This line could use = default syntax. This would also reduce the number of lines in the associated cpp. https://github.com/microsoft/Terminal/blob/bf460ab7feed7e203f1c5cf1bc06a1c732742870/src/terminal/input/terminalInput.hpp#L26<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FTerminal%2Fblob%2Fbf460ab7feed7e203f1c5cf1bc06a1c732742870%2Fsrc%2Fterminal%2Finput%2FterminalInput.hpp%23L26&data=01%7C01%7Cduhowett%40microsoft.com%7C04c35bbc97b9461602b708d6d5970abd%7C72f988bf86f141af91ab2d7cd011db47%7C1&sdata=H6ih1sI%2B7kBB9zz2oCmdVqR5ITumQ%2BfDrfu%2FA8CH1JY%3D&reserved=0> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FTerminal%2Fissues%2F685&data=01%7C01%7Cduhowett%40microsoft.com%7C04c35bbc97b9461602b708d6d5970abd%7C72f988bf86f141af91ab2d7cd011db47%7C1&sdata=r9ZC3yMUHZLAbhs8PJJ%2Bl1Ss2Py%2BB%2FZTA59TkHYU5V0%3D&reserved=0>, or mute the thread<https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADNHLGW5ROOZCAES27BZAPTPUXZOTANCNFSM4HMGTLTA&data=01%7C01%7Cduhowett%40microsoft.com%7C04c35bbc97b9461602b708d6d5970abd%7C72f988bf86f141af91ab2d7cd011db47%7C1&sdata=1zzpQbZuJbRdwSCtPUH66JRJDwLxqyQnl44cGA0%2BzpQ%3D&reserved=0>.
Author
Owner

@zadjii-msft commented on GitHub (Jul 15, 2021):

Oh weird, this was fixed in #690, just never marked as closed.

@zadjii-msft commented on GitHub (Jul 15, 2021): Oh weird, this was fixed in #690, just never marked as closed.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#966