Skip to content

Commit d72a413

Browse files
committed
Small fixes for optionals
1 parent 8245e85 commit d72a413

File tree

3 files changed

+19
-3
lines changed

3 files changed

+19
-3
lines changed

Samples/JavaKitSampleApp/Sources/JavaKitExample/JavaKitExample.swift

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,13 @@ extension HelloSwift: HelloSwiftNativeMethods {
8989
let text: JavaString? = helper.textOptional
9090
let value: String? = helper.getValueOptional(Optional<JavaString>.none)
9191
let textFunc: JavaString? = helper.getTextOptional()
92+
let doubleOpt: Double? = helper.valOptional
93+
let longOpt: Int64? = helper.fromOptional(21 as Int32?)
94+
print("Optional text = \(text)")
95+
print("Optional string value = \(value)")
96+
print("Optional text function returned \(textFunc)")
97+
print("Optional double function returned \(doubleOpt)")
98+
print("Optional long function returned \(longOpt)")
9299
}
93100

94101
@JavaMethod

Samples/JavaKitSampleApp/Sources/JavaKitExample/com/example/swift/ThreadSafeHelperClass.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,18 +15,27 @@
1515
package com.example.swift;
1616

1717
import java.util.Optional;
18+
import java.util.OptionalLong;
19+
import java.util.OptionalInt;
20+
import java.util.OptionalDouble;
1821

1922
@ThreadSafe
2023
public class ThreadSafeHelperClass {
2124
public ThreadSafeHelperClass() { }
2225

2326
public Optional<String> text = Optional.of("");
2427

28+
public final OptionalDouble val = OptionalDouble.of(2);
29+
2530
public String getValue(Optional<String> name) {
2631
return name.orElse("");
2732
}
2833

2934
public Optional<String> getText() {
3035
return text;
3136
}
37+
38+
public OptionalLong from(OptionalInt value) {
39+
return OptionalLong.of(value.getAsInt());
40+
}
3241
}

Sources/Java2SwiftLib/StringExtras.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,11 @@ extension String {
4141
}
4242

4343
public func optionalWrappedType() -> String? {
44-
if self == "JavaOptionalInt!" {
44+
if self == "JavaOptionalInt!" || self == "JavaOptionalInt?" {
4545
return "Int32?"
46-
} else if self == "JavaOptionalLong!" {
46+
} else if self == "JavaOptionalLong!" || self == "JavaOptionalLong?" {
4747
return "Int64?"
48-
} else if self == "JavaOptionalDouble!" {
48+
} else if self == "JavaOptionalDouble!" || self == "JavaOptionalDouble?" {
4949
return "Double?"
5050
} else if hasPrefix("JavaOptional<") {
5151
let offsetCount = "JavaOptional<".count

0 commit comments

Comments
 (0)