Hi Alan,
http://cr.openjdk.java.net/~clanger/webrevs/8213031.10/ I see your other mail with v11 so I switch to that version.
Right. 😊
Ok, makes sense. I've updated the coding such that the zip's file owner would be the default owner, in case it can be retrieved. I think this is mostly right now. One thing to check is that you are catching UOE whereas it's the PAE's cause that might be UOE.
It's alright as it is. If you look at the source of AccessController.doPrivileged for a PrivilegedExceptionAction, you will see that RuntimeExceptions are caught and rethrown, not yielding a PAE. So I must catch UOE here explicitly because it's a RuntimeException.
: I have updated the doc in module-info.java quite a bit. Please check.
Is it time to work on the CSR now? How shall we proceed there?
The table items in L119-150 look fine, we just need to avoid really long lines One minor comment on L123 is that it might be clearer if you drop "created" from the sentence.
L48-78 is a "wall of text" and links that I don't think will be easy for most developers to read. Can I provide suggested wording for this part of the spec? I'm just thinking that an alternative wording might help avoid too much iteration on this.
Sure, you're very welcome 😊 So, I'll be waiting for your input on that. Thanks Christoph