Skip to content

Commit e5038f1

Browse files
committed
[NFC] Rename types to mention optional
(cherry picked from commit 9b0ab90)
1 parent 1dda72d commit e5038f1

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

test/Interop/Cxx/class/Inputs/member-variables.h

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,23 +27,23 @@ struct HasZeroSizedField {
2727
void set_c(short c) { this->c = c; }
2828
};
2929

30-
struct ReuseFieldPadding {
30+
struct ReuseOptionalFieldPadding {
3131
[[no_unique_address]] std::optional<int> a = {2};
3232
char c;
3333
char get_c() const { return c; }
3434
void set_c(char c) { this->c = c; }
35-
int offset() const { return offsetof(ReuseFieldPadding, c); }
35+
int offset() const { return offsetof(ReuseOptionalFieldPadding, c); }
3636
std::optional<int> getOptional() { return a; }
3737
};
3838

3939
using OptInt = std::optional<int>;
4040

41-
struct ReuseFieldPaddingWithTypedef {
41+
struct ReuseOptionalFieldPaddingWithTypedef {
4242
[[no_unique_address]] OptInt a;
4343
char c;
4444
char get_c() const { return c; }
4545
void set_c(char c) { this->c = c; }
46-
int offset() const { return offsetof(ReuseFieldPadding, c); }
46+
int offset() const { return offsetof(ReuseOptionalFieldPadding, c); }
4747
};
4848

4949

test/Interop/Cxx/class/zero-sized-field.swift

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,12 @@ FieldsTestSuite.test("Zero sized field") {
2424
expectEqual(s.b.getNum(), 42)
2525
}
2626

27-
FieldsTestSuite.test("Field padding reused") {
28-
var s = ReuseFieldPadding()
27+
FieldsTestSuite.test("Optional field padding reused") {
28+
var s = ReuseOptionalFieldPadding()
2929
let opt = s.getOptional()
3030
expectEqual(Int(opt.pointee), 2)
3131
s.c = 5
32-
expectEqual(Int(s.offset()), MemoryLayout<ReuseFieldPadding>.offset(of: \.c)!)
32+
expectEqual(Int(s.offset()), MemoryLayout<ReuseOptionalFieldPadding>.offset(of: \.c)!)
3333
expectEqual(s.c, 5)
3434
expectEqual(s.get_c(), 5)
3535
s.set_c(6)
@@ -40,10 +40,10 @@ FieldsTestSuite.test("Field padding reused") {
4040
expectEqual(s2.get_c(), 6)
4141
}
4242

43-
FieldsTestSuite.test("Typedef'd field padding reused") {
44-
var s = ReuseFieldPaddingWithTypedef()
43+
FieldsTestSuite.test("Typedef'd optional field padding reused") {
44+
var s = ReuseOptionalFieldPaddingWithTypedef()
4545
s.c = 5
46-
expectEqual(Int(s.offset()), MemoryLayout<ReuseFieldPadding>.offset(of: \.c)!)
46+
expectEqual(Int(s.offset()), MemoryLayout<ReuseOptionalFieldPadding>.offset(of: \.c)!)
4747
expectEqual(s.c, 5)
4848
expectEqual(s.get_c(), 5)
4949
s.set_c(6)

0 commit comments

Comments
 (0)