RFR (3rd): 8023524: Mechanism to dump generated lambda classes / log lambda code generation
mandy.chung at oracle.com
Thu Sep 26 23:01:40 UTC 2013
On 9/25/2013 12:37 AM, Henry Jen wrote:
> Please review the update webrev at
The doPrivileged block looks okay. It'd be good to limit privileges
by calling the doPrivileged method with a specific set of Permissions.
In this case, it would be FilePermission("<<ALL FILES>>", "write").
If "jdk.internal.lambda.dumpProxyClasses" property is set with empty
value, would it be better to default to CWD? ProxyClassesDumper currently
fails with non-existent directory.
ProxyClassesDumper - you may want to use java.nio.file.Path and
Files. I also suggest to follow the convention of classfile pathname
and generate it in the directories reflecting the package name i.e.
instead of DIR/com.example.TestLambda$$Lambda$1.class,
For escaping sequence, as Peter pointed out, I presume you can
consider using ParseUtil.encodePath but I am not sure how importance
it has to encode the path as this is for debugging purpose.
No strong opinion in this one.
More information about the core-libs-dev