[jextract] add support for throwing functions in JNI mode #277
Conversation
Sources/JExtractSwiftLib/SwiftTypes/SwiftFunctionSignature.swift
Outdated
Show resolved
Hide resolved
| } catch { | ||
| environment.throwAsException(error) | ||
| return Int64.jniPlaceholderValue | ||
| } |
|
|
||
| enum SwiftEffectSpecifier: Equatable { | ||
| case `throws` | ||
| } |
There was a problem hiding this comment.
I think that's okey, in swift-syntax they're just as a keyword afair -- any opinion about such enum @rintaro ?
There was a problem hiding this comment.
The enum look okay to me for now. We should support typed throw some day, but not now.
|
Looks nice! When you enable previously-unsupported thing in |
| } | ||
| case .accessors(let accessors): | ||
| if let getter = accessors.first(where: { $0.accessorSpecifier.tokenKind == .keyword(.set) }) { | ||
| effectSpecifiers = Self.effectSpecifiers(from: getter) |
There was a problem hiding this comment.
We should do this only if isSet is false.
| switch binding.accessorBlock?.accessors { | ||
| case .getter(let getter): | ||
| if let getter = getter.as(AccessorDeclSyntax.self) { | ||
| effectSpecifiers = Self.effectSpecifiers(from: getter) |
There was a problem hiding this comment.
AccessorBlockSyntax.Accessors.getter can only contain CodeBlockItemListSyntax so this should be unreachable.
|
Disabling |
Adds support for generating throwing functions in JNI mode