Set Foundation.Data.SubSequence to be RangeReplaceable

gribozavr · Dave Abrahams · commit 32184b47b012 · 2016-09-28T18:36:15.000-07:00
RangeReplaceableCollection.SubSequence should be a
RangeReplaceableCollection.  Data implements RangeReplaceableCollection,
but sets SubSequence to MutableRandomAccessSlice that is not a
RangeReplaceableCollection.

This commit changes Foundation.Data.SubSequence to
MutableRangeReplaceableRandomAccessSlice.

Fixes rdar://problem/28330713.
diff --git a/Foundation/Data.swift b/Foundation/Data.swift
@@ -652,9 +652,9 @@ public struct Data : ReferenceConvertible, Equatable, Hashable, RandomAccessColl
         }
     }
     
-    public subscript(bounds: Range<Index>) -> MutableRandomAccessSlice<Data> {
+    public subscript(bounds: Range<Index>) -> MutableRangeReplaceableRandomAccessSlice<Data> {
         get {
-            return MutableRandomAccessSlice(base: self, bounds: bounds)
+            return MutableRangeReplaceableRandomAccessSlice(base: self, bounds: bounds)
         }
         set {
             replaceSubrange(bounds, with: newValue.base)

Original file line number	Diff line number	Diff line change
`@@ -652,9 +652,9 @@ public struct Data : ReferenceConvertible, Equatable, Hashable, RandomAccessColl`
`652`	`652`	`}`
`653`	`653`	`}`
`654`	`654`
`655`		`- public subscript(bounds: Range<Index>) -> MutableRandomAccessSlice<Data> {`
	`655`	`+ public subscript(bounds: Range<Index>) -> MutableRangeReplaceableRandomAccessSlice<Data> {`
`656`	`656`	`get {`
`657`		`- return MutableRandomAccessSlice(base: self, bounds: bounds)`
	`657`	`+ return MutableRangeReplaceableRandomAccessSlice(base: self, bounds: bounds)`
`658`	`658`	`}`
`659`	`659`	`set {`
`660`	`660`	`replaceSubrange(bounds, with: newValue.base)`