Skip to content

Commit c94b250

Browse files
committed
small cleanups
1 parent e7c8619 commit c94b250

File tree

1 file changed

+1
-18
lines changed

1 file changed

+1
-18
lines changed

Sources/JExtractSwiftLib/JNI/JNISwift2JavaGenerator+JavaTranslation.swift

Lines changed: 1 addition & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -490,15 +490,6 @@ extension JNISwift2JavaGenerator {
490490
) {
491491
switch mode {
492492
case .completableFuture:
493-
// let supplyAsyncBodyConversion: JavaNativeConversionStep = if result.javaType.isVoid {
494-
// .aggregate([
495-
// .print(result.conversion),
496-
// .null
497-
// ])
498-
// } else {
499-
// result.conversion
500-
// }
501-
502493
// Update translated function
503494

504495
let nativeFutureType = JavaType.completableFuture(nativeFunctionSignature.result.javaType)
@@ -986,14 +977,9 @@ extension JNISwift2JavaGenerator {
986977
/// Access a member of the value
987978
indirect case replacingPlaceholder(JavaNativeConversionStep, placeholder: String)
988979

989-
/// `return value`
990-
indirect case `return`(JavaNativeConversionStep)
991-
992980
/// `(args) -> { return body; }`
993981
indirect case lambda(args: [String] = [], body: JavaNativeConversionStep)
994982

995-
case null
996-
997983
/// Prints the conversion step, ignoring the output.
998984
indirect case print(JavaNativeConversionStep)
999985

@@ -1139,7 +1125,7 @@ extension JNISwift2JavaGenerator {
11391125
/// Whether the conversion uses SwiftArena.
11401126
var requiresSwiftArena: Bool {
11411127
switch self {
1142-
case .placeholder, .constant, .isOptionalPresent, .combinedName, .null:
1128+
case .placeholder, .constant, .isOptionalPresent, .combinedName:
11431129
return false
11441130

11451131
case .constructSwiftValue, .wrapMemoryAddressUnsafe:
@@ -1184,9 +1170,6 @@ extension JNISwift2JavaGenerator {
11841170
case .replacingPlaceholder(let inner, _):
11851171
return inner.requiresSwiftArena
11861172

1187-
case .return(let inner):
1188-
return inner.requiresSwiftArena
1189-
11901173
case .lambda(_, let body):
11911174
return body.requiresSwiftArena
11921175

0 commit comments

Comments
 (0)