Skip to content

Commit dcfa442

Browse files
author
Pouya Yarandi
committed
Rename merge(from:) to merge(with:)
1 parent 9e15359 commit dcfa442

File tree

2 files changed

+8
-8
lines changed

2 files changed

+8
-8
lines changed

Sources/SwiftProtobuf/Message+FieldMask.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ extension Message {
5757
/// - source: Message should be merged to the original one.
5858
/// - fieldMask: FieldMask specifies which fields should be merged.
5959
public mutating func merge(
60-
from source: Self,
60+
with source: Self,
6161
fieldMask: Google_Protobuf_FieldMask,
6262
mergeOption: Google_Protobuf_FieldMask.MergeOptions = .init()
6363
) throws {
@@ -97,7 +97,7 @@ extension Message where Self: Equatable, Self: _ProtoNameProviding {
9797
}
9898
var tmp = Self(removingAllFieldsOf: self)
9999
do {
100-
try tmp.merge(from: self, fieldMask: fieldMask)
100+
try tmp.merge(with: self, fieldMask: fieldMask)
101101
let changed = tmp != self
102102
self = tmp
103103
return changed

Tests/SwiftProtobufTests/Test_FieldMask.swift

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -124,12 +124,12 @@ final class Test_FieldMask: XCTestCase, PBTestHelpers {
124124
}
125125

126126
// Checks nested message merge
127-
try message.merge(from: secondMessage, fieldMask: .init(protoPaths: "optional_nested_message.bb"))
127+
try message.merge(with: secondMessage, fieldMask: .init(protoPaths: "optional_nested_message.bb"))
128128
XCTAssertEqual(message.optionalInt32, 1)
129129
XCTAssertEqual(message.optionalNestedMessage.bb, 3)
130130

131131
// Checks primitive type merge
132-
try message.merge(from: secondMessage, fieldMask: .init(protoPaths: "optional_int32"))
132+
try message.merge(with: secondMessage, fieldMask: .init(protoPaths: "optional_int32"))
133133
XCTAssertEqual(message.optionalInt32, 2)
134134
XCTAssertEqual(message.optionalNestedMessage.bb, 3)
135135
}
@@ -147,13 +147,13 @@ final class Test_FieldMask: XCTestCase, PBTestHelpers {
147147
let fieldMask = Google_Protobuf_FieldMask(protoPaths: ["repeated_int32"])
148148

149149
// Checks without replacing repeated fields
150-
try message.merge(from: secondMessage, fieldMask: fieldMask)
150+
try message.merge(with: secondMessage, fieldMask: fieldMask)
151151
XCTAssertEqual(message.repeatedInt32, [1, 2, 3, 4])
152152

153153
// Checks with replacing repeated fields
154154
var options = Google_Protobuf_FieldMask.MergeOptions()
155155
options.replaceRepeatedFields = true
156-
try message.merge(from: secondMessage, fieldMask: fieldMask, mergeOption: options)
156+
try message.merge(with: secondMessage, fieldMask: fieldMask, mergeOption: options)
157157
XCTAssertEqual(message.repeatedInt32, [3, 4])
158158
}
159159

@@ -170,13 +170,13 @@ final class Test_FieldMask: XCTestCase, PBTestHelpers {
170170
let fieldMask = Google_Protobuf_FieldMask(protoPaths: ["map_int32_string"])
171171

172172
// Checks without replacing repeated fields
173-
try message.merge(from: secondMessage, fieldMask: fieldMask)
173+
try message.merge(with: secondMessage, fieldMask: fieldMask)
174174
XCTAssertEqual(message.mapInt32String, [1: "a", 2: "b"])
175175

176176
// Checks with replacing repeated fields
177177
var options = Google_Protobuf_FieldMask.MergeOptions()
178178
options.replaceRepeatedFields = true
179-
try message.merge(from: secondMessage, fieldMask: fieldMask, mergeOption: options)
179+
try message.merge(with: secondMessage, fieldMask: fieldMask, mergeOption: options)
180180
XCTAssertEqual(message.mapInt32String, [2: "b"])
181181
}
182182

0 commit comments

Comments
 (0)