Thanks for doing this Max. The syntax looks fine.
Just one question. Do you think it is better to specify each socket
option literally in the tool
as you have done (ie. the only supported NetworkPermission
is SO_FLOW_SLA with this change) or allow users to type in the option
name as free-form text
I guess you can always edit policy files manually, but if more socket
options get added
in future, then we will need to update the tool again.
Michael
On 26/08/14 01:58, Wang Weijun wrote:
Hi Michael
Please review the code change at
http://cr.openjdk.java.net/~weijun/8055901/webrev.00/
With this change, the policy tool can generate a file like
grant {
permission jdk.net.NetworkPermission "setOption.SO_FLOW_SLA";
};
Please confirm the grammar is correct.
BTW, I also add some missing targets in a nearby permission type.
Noreg-trivial.
Thanks
Max