Avoid unnecessary SetLastError on PInvokes (dotnet/corefx#36544)

These console PInvokes are used in nearly every app. Omitting unnecessary SetLastError
will save us from generating marshalling stub for them.

Signed-off-by: dotnet-bot <[email protected]>

Author: jkotas

src/System.Private.CoreLib/System.Private.CoreLib.csproj

@@ -271,6 +271,7 @@
     <Compile Include="$(BclSourcesRoot)\System\ValueType.cs" />
     <Compile Include="$(BclSourcesRoot)\System\WeakReference.cs" />
     <Compile Include="$(BclSourcesRoot)\System\WeakReferenceOfT.cs" />
+    <Compile Include="shared\Interop\Windows\Kernel32\Interop.HandleTypes.cs" />
     <Compile Include="shared\Interop\Windows\Kernel32\Interop.GetStdHandle.cs" />
     <Compile Include="shared\Interop\Windows\Kernel32\Interop.LocalAlloc.cs" />
     <Compile Include="shared\Interop\Windows\Kernel32\Interop.QueryUnbiasedInterruptTime.cs" />

src/System.Private.CoreLib/shared/Interop/Windows/Kernel32/Interop.GetStdHandle.cs

@@ -9,11 +9,7 @@ internal partial class Interop
 {
     internal partial class Kernel32
     {
-        internal const int STD_INPUT_HANDLE = -10;
-        internal const int STD_OUTPUT_HANDLE = -11;
-        internal const int STD_ERROR_HANDLE = -12;
-
-        [DllImport(Libraries.Kernel32, SetLastError = true)]
-        internal static extern IntPtr GetStdHandle(int nStdHandle);
+        [DllImport(Libraries.Kernel32)]
+        internal static extern IntPtr GetStdHandle(int nStdHandle);  // param is NOT a handle, but it returns one!
     }
 }

src/System.Private.CoreLib/shared/Interop/Windows/Kernel32/Interop.HandleTypes.cs

@@ -0,0 +1,16 @@
+// Licensed to the .NET Foundation under one or more agreements.
+// The .NET Foundation licenses this file to you under the MIT license.
+// See the LICENSE file in the project root for more information.
+
+internal partial class Interop
+{
+    internal partial class Kernel32
+    {
+        internal partial class HandleTypes
+        {
+            internal const int STD_INPUT_HANDLE = -10;
+            internal const int STD_OUTPUT_HANDLE = -11;
+            internal const int STD_ERROR_HANDLE = -12;
+        }
+    }
+}

src/System.Private.CoreLib/src/Internal/Console.cs

@@ -16,7 +16,7 @@ namespace Internal
     public static class Console
     {
         private static readonly SafeFileHandle _outputHandle =
-            new SafeFileHandle(Interop.Kernel32.GetStdHandle(Interop.Kernel32.STD_OUTPUT_HANDLE), ownsHandle: false);
+            new SafeFileHandle(Interop.Kernel32.GetStdHandle(Interop.Kernel32.HandleTypes.STD_OUTPUT_HANDLE), ownsHandle: false);
 
         public static unsafe void Write(string s)
         {