maxmustermann/terminal
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Remove unneeded whitespace (#5162)

Browse source 
<!-- Enter a brief description/summary of your PR here. What does it fix/what does it change/how was it tested (even manually, if necessary)? -->
## Summary of the Pull Request
Every single time a PR is run, there are a bunch of warnings about whitespace in the .cs files, so I ran the code format on those files, without changing their contents, so it won't be flagged anymore.
<!-- Other than the issue solved, is this relevant to any other issues/existing PRs? --> 
## References

<!-- Please review the items on the PR checklist before submitting-->
## PR Checklist
* [X] CLA signed. If not, go over [here](https://cla.opensource.microsoft.com/microsoft/Terminal) and sign the CLA
* [X] Tests added/passed

<!-- Provide a more detailed description of the PR, other things fixed or any additional comments/features here -->
## Detailed Description of the Pull Request / Additional comments

<!-- Describe how you validated the behavior. Add automated tests wherever possible, but list manual validation steps taken as well -->
## Validation Steps Performed
Ran the code-format utility on the .cs files
This commit is contained in:
pi1024e 2020-03-30 10:33:32 -04:00 committed by GitHub
parent 28d108bf32
commit 2872f147f8
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
12 changed files with 25 additions and 25 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/cascadia/WpfTerminalControl/NativeMethods.cs
View file

@ -38,7 +38,7 @@ namespace Microsoft.Terminal.Wpf
WM_MOUSEACTIVATE = 0x0021,
WM_GETOBJECT = 0x003D,
/// <summary>
/// The WM_WINDOWPOSCHANGED message is sent to a window whose size, position, or place in the Z order has changed as a result of a call to the SetWindowPos function or another window-management function.
/// </summary>

4
src/cascadia/WpfTerminalControl/TerminalContainer.cs
View file

@ -26,7 +26,7 @@ namespace Microsoft.Terminal.Wpf
private DispatcherTimer blinkTimer;
private NativeMethods.ScrollCallback scrollCallback;
private NativeMethods.WriteCallback writeCallback;
/// <summary>
/// Initializes a new instance of the <see cref="TerminalContainer"/> class.
/// </summary>
@ -35,7 +35,7 @@ namespace Microsoft.Terminal.Wpf
this.MessageHook += this.TerminalContainer_MessageHook;
this.GotFocus += this.TerminalContainer_GotFocus;
this.Focusable = true;
var blinkTime = NativeMethods.GetCaretBlinkTime();
if (blinkTime != uint.MaxValue)

2
src/cascadia/WpfTerminalControl/TerminalControl.xaml.cs
View file

@ -9,7 +9,7 @@ namespace Microsoft.Terminal.Wpf
using System.Windows.Controls;
using System.Windows.Input;
using System.Windows.Media;
/// <summary>
/// A basic terminal control. This control can receive and render standard VT100 sequences.
/// </summary>

2
src/host/ft_uia/AccessibilityTests.cs
View file

@ -481,7 +481,7 @@ namespace Conhost.UIA.Tests
TextPatternRange testRange = visibleRanges.First().Clone();
// assumes that range is a line range at the top of the screen buffer
Action<TextPatternRange> testTopBoundary = delegate(TextPatternRange range)
Action<TextPatternRange> testTopBoundary = delegate (TextPatternRange range)
{
// the first visible range is at the top of the screen
// buffer, we shouldn't be able to move the starting endpoint up

2
src/host/ft_uia/Common/ShortcutHelper.cs
View file

@ -31,7 +31,7 @@ namespace Conhost.UIA.Tests.Common
~ShortcutHelper()
{
this.Dispose(false);
this.Dispose(false);
}
public void Dispose()

2
src/host/ft_uia/Elements/TabBase.cs
View file

@ -52,7 +52,7 @@ namespace Conhost.UIA.Tests.Elements
tab.Click();
Globals.WaitForTimeout();
this.PopulateItemsOnNavigate(this.propDialog.PropWindow);
}

2
src/host/ft_uia/Elements/Tabs.cs
View file

@ -33,7 +33,7 @@ namespace Conhost.UIA.Tests.Elements
this.tabs.Add(new FontTab(this.propDialog));
this.tabs.Add(new LayoutTab(this.propDialog));
this.tabs.Add(new ColorsTab(this.propDialog));
}
private Tabs()

2
src/host/ft_uia/Elements/ViewportArea.cs
View file

@ -105,7 +105,7 @@ namespace Conhost.UIA.Tests.Elements
app.UIRoot.SendKeys(Keys.Escape);
this.state = ViewportStates.Normal;
}
public void EnterMode(ViewportStates state)
{
if (state == ViewportStates.Normal)

2
src/host/ft_uia/MouseWheelTests.cs
View file

@ -111,7 +111,7 @@ namespace Conhost.UIA.Tests
default:
throw new NotSupportedException();
}
afterScroll = app.GetScreenBufferInfo();
switch (dir)

8
src/host/ft_uia/VirtualTerminalTests.cs
View file

@ -57,7 +57,7 @@ namespace Conhost.UIA.Tests
reg.BackupRegistry(); // we're going to modify the virtual terminal state for this, so back it up first.
VersionSelector.SetConsoleVersion(reg, ConsoleVersion.V2);
reg.SetDefaultValue(VIRTUAL_TERMINAL_KEY_NAME, VIRTUAL_TERMINAL_ON_VALUE);
bool haveVtAppPath = !string.IsNullOrEmpty(vtAppLocation);
Verify.IsTrue(haveVtAppPath, "Ensure that we passed in the location to VtApp.exe");
@ -114,12 +114,12 @@ namespace Conhost.UIA.Tests
Log.Comment("Move cursor to the middle-ish");
Point cursorExpected = new Point();
// H is at 5, 1. VT coords are 1-based and buffer is 0-based so adjust.
cursorExpected.Y = 5 - 1;
cursorExpected.Y = 5 - 1;
cursorExpected.X = 1 - 1;
app.UIRoot.SendKeys("H");
// Move to middle-ish from here. 10 Bs and 10 Cs should about do it.
for (int i=0; i < 10; i++)
for (int i = 0; i < 10; i++)
{
app.UIRoot.SendKeys("BC");
cursorExpected.Y++;
@ -715,7 +715,7 @@ namespace Conhost.UIA.Tests
}
delegate char GetExpectedChar(int rowId, int colId, int height, int width);
private static void ScreenFillHelper(CmdApp app, ViewportArea area, IntPtr hConsole)
{
Log.Comment("Fill screen with junk");

8
src/host/ft_uia/WinEventTests.cs
View file

@ -212,7 +212,7 @@ namespace Conhost.UIA.Tests
[TestMethod]
public void TestSelection()
{
RunTest(TestSelectionImpl);
RunTest(TestSelectionImpl);
}
private void TestSelectionImpl(CmdApp app, ViewportArea area, IntPtr hConsole, WinCon.CONSOLE_SCREEN_BUFFER_INFO_EX sbiex, Queue<EventData> expected, WinCon.CONSOLE_SCREEN_BUFFER_INFO_EX sbiexOriginal)
@ -295,7 +295,7 @@ namespace Conhost.UIA.Tests
[TestMethod]
public void TestLaunchAndExitChild()
{
RunTest(TestLaunchAndExitChildImpl);
RunTest(TestLaunchAndExitChildImpl);
}
private void TestLaunchAndExitChildImpl(CmdApp app, ViewportArea area, IntPtr hConsole, WinCon.CONSOLE_SCREEN_BUFFER_INFO_EX sbiex, Queue<EventData> expected, WinCon.CONSOLE_SCREEN_BUFFER_INFO_EX sbiexOriginal)
@ -357,11 +357,11 @@ namespace Conhost.UIA.Tests
{
RunTest(TestScrollByWheelImpl);
}
private void TestScrollByWheelImpl(CmdApp app, ViewportArea area, IntPtr hConsole, WinCon.CONSOLE_SCREEN_BUFFER_INFO_EX sbiex, Queue<EventData> expected, WinCon.CONSOLE_SCREEN_BUFFER_INFO_EX sbiexOriginal)
{
int rowsPerScroll = app.GetRowsPerScroll();
int scrollDelta;
int scrollDelta;
// A. Scroll down.
{

14
src/tools/vtapp/Program.cs
View file

@ -13,14 +13,14 @@ namespace VTApp
{
class Program
{
static string CSI = ((char)0x1b)+"[";
static string CSI = ((char)0x1b) + "[";
static void Main(string[] args)
{
Console.WindowHeight = 25;
Console.BufferHeight = 9000;
Console.WindowWidth = 80;
Console.BufferWidth = 80;
Console.WriteLine("VT Tester");
while (true)
@ -271,11 +271,11 @@ namespace VTApp
Console.Write("49m");
break;
case '<':
Console.Write('\xD'); // carriage return \r
break;
Console.Write('\xD'); // carriage return \r
break;
case '>':
Console.Write('\xA'); // line feed/new line \n
break;
Console.Write('\xA'); // line feed/new line \n
break;
case '`':
Console.Write("z");
break;
@ -385,7 +385,7 @@ namespace VTApp
for (int j = 0; j < 80; j++)
{
if (j == 0)
Console.Write(i%10);
Console.Write(i % 10);
else
Console.Write("Z");
}