Review comments.

Author: ktopley-apple

src/swift/Data.swift

@@ -72,7 +72,7 @@ public struct DispatchData : RandomAccessCollection {
 	///
 	/// - parameter bytes: A buffer pointer containing the data.
 	/// - parameter deallocator: Specifies the mechanism to free the indicated buffer.
-	@available(swift, deprecated: 4, message: "Use init(bytes: UnsafeRawBufferPointer, deallocater: Deallocator) instead")
+	@available(swift, deprecated: 4, message: "Use init(bytesNoCopy: UnsafeRawBufferPointer, deallocater: Deallocator) instead")
 	public init(bytesNoCopy bytes: UnsafeBufferPointer<UInt8>, deallocator: Deallocator = .free) {
 		let (q, b) = deallocator._deallocator
 		let d = bytes.baseAddress == nil ? _swift_dispatch_data_empty()
@@ -200,10 +200,11 @@ public struct DispatchData : RandomAccessCollection {
 
 	/// Copy the contents of the data to a pointer.
 	///
-	/// - parameter pointer: A pointer to the buffer you wish to copy the bytes into.
+	/// - parameter pointer: A pointer to the buffer you wish to copy the bytes into. The buffer must be large
+	///	enough to hold `count` bytes.
 	/// - parameter count: The number of bytes to copy.
-	/// - warning: This method does not verify that the contents at pointer have enough space to hold `count` bytes.
 	public func copyBytes(to pointer: UnsafeMutableRawBufferPointer, count: Int) {
+		assert(count <= pointer.count, "Buffer too small to copy \(count) bytes")
 		guard pointer.baseAddress != nil else { return }
 		_copyBytesHelper(to: pointer.baseAddress!, from: 0..<count)
 	}
@@ -220,10 +221,11 @@ public struct DispatchData : RandomAccessCollection {
 
 	/// Copy a subset of the contents of the data to a pointer.
 	///
-	/// - parameter pointer: A pointer to the buffer you wish to copy the bytes into.
+	/// - parameter pointer: A pointer to the buffer you wish to copy the bytes into. The buffer must be large
+	///	enough to hold `count` bytes.
 	/// - parameter range: The range in the `Data` to copy.
-	/// - warning: This method does not verify that the contents at pointer have enough space to hold the required number of bytes.
 	public func copyBytes(to pointer: UnsafeMutableRawBufferPointer, from range: CountableRange<Index>) {
+		assert(range.count <= pointer.count, "Buffer too small to copy \(range.count) bytes")
 		guard pointer.baseAddress != nil else { return }
 		_copyBytesHelper(to: pointer.baseAddress!, from: range)
 	}