On Fri, 18 Dec 2020 22:22:33 GMT, Sergey Bylokhov <s...@openjdk.org> wrote:
> > > I guess it should work already in the JNF case? The java exception should > > > be wrapped by the NSException in the JNFCallObjectMethod/etc and > > > unwrapped in the JNF_COCOA_ENTER before exists to the java method? > > > > > > That is the pattern I know is there today and I don't think is needed. > > It can be done but I don't think it needed. I'd prefer to see if we ever > > have such a situation and deal with it "locally" > > at the call site. > > No, it is quite useful, and it will be good to preserve it. It is possible to > work "locally" when the call site if come from java, but not if we are in the > infinite native loop. an NS exception on the appkit will terminate the app - if unhandled - and probably will even if handled so I am not sure what is better about that > > > > > So I believe we may end up more (theoretically) robust with my changes > > > > than we were before. > > > > Perhaps they weren't a result of a JNF* call but still it seems > > > > improved. > > > > I am not sure about that. > > > > I am looking at JNI warnings - not just guessing. > > I am looking at a new code that is not as good as before. I guess we disagree. I think this is better and simpler. ------------- PR: https://git.openjdk.java.net/jdk/pull/1679