Address PR feedback

pjanotti · Jan 20, 2018 · 94e3346 · 94e3346
1 parent a4c639f
commit 94e3346
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/Common/tests/System/Buffers/NativeOwnedMemory.cs b/src/Common/tests/System/Buffers/NativeOwnedMemory.cs
@@ -55,7 +55,7 @@ protected override bool IsRetained
 
         public override unsafe MemoryHandle Pin(int offset = 0)
         {
-            if (offset < 0 || (_length > 0 && offset >= _length)) throw new ArgumentOutOfRangeException(nameof(offset));
+            if (offset < 0 || offset > _length) throw new ArgumentOutOfRangeException(nameof(offset));
             void* pointer = (void*)((byte*)_ptr + offset);
             return new MemoryHandle(this, pointer);
         }

diff --git a/src/System.Memory/tests/Memory/CustomMemoryForTest.cs b/src/System.Memory/tests/Memory/CustomMemoryForTest.cs
@@ -44,7 +44,7 @@ public override MemoryHandle Pin(int offset = 0)
             unsafe
             {
                 Retain();
-                if (offset < 0 || (_array.Length > 0 && offset >= _array.Length)) throw new ArgumentOutOfRangeException(nameof(offset));
+                if (offset < 0 || offset > _array.Length) throw new ArgumentOutOfRangeException(nameof(offset));
                 var handle = GCHandle.Alloc(_array, GCHandleType.Pinned);
                 return new MemoryHandle(this, Unsafe.Add<byte>((void*)handle.AddrOfPinnedObject(), offset), handle);
             }