Improve/fix SerialStream.Dispose (dotnet/corefxdotnet/coreclr#33221)
authorKrzysztof Wicher <mordotymoja@gmail.com>
Mon, 5 Nov 2018 18:48:53 +0000 (10:48 -0800)
committerJan Kotas <jkotas@microsoft.com>
Mon, 5 Nov 2018 23:34:39 +0000 (15:34 -0800)
* Improve/fix SerialStream.Dispose

* Review feedback

* remove unused using statement

* Use ToFileDescriptor

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Commit migrated from https://github.com/dotnet/coreclr/commit/0eb2dbbc3cd45b2a118aa44ac05bac9b8ca1eec4

src/libraries/System.Private.CoreLib/src/Interop/Unix/System.Native/Interop.Read.cs
src/libraries/System.Private.CoreLib/src/Interop/Unix/System.Native/Interop.Write.cs

index 1be5e78..233feab 100644 (file)
@@ -20,6 +20,6 @@ internal static partial class Interop
         /// Note - on fail. the position of the stream may change depending on the platform; consult man 2 read for more info
         /// </returns>
         [DllImport(Libraries.SystemNative, EntryPoint = "SystemNative_Read", SetLastError = true)]
-        internal static extern unsafe int Read(SafeFileHandle fd, byte* buffer, int count);
+        internal static extern unsafe int Read(SafeHandle fd, byte* buffer, int count);
     }
 }
index 0636615..fb06d46 100644 (file)
@@ -19,7 +19,7 @@ internal static partial class Interop
         /// Returns the number of bytes written on success; otherwise, returns -1 and sets errno
         /// </returns>
         [DllImport(Libraries.SystemNative, EntryPoint = "SystemNative_Write", SetLastError = true)]
-        internal static extern unsafe int Write(SafeFileHandle fd, byte* buffer, int bufferSize);
+        internal static extern unsafe int Write(SafeHandle fd, byte* buffer, int bufferSize);
 
         [DllImport(Libraries.SystemNative, EntryPoint = "SystemNative_Write", SetLastError = true)]
         internal static extern unsafe int Write(int fd, byte* buffer, int bufferSize);