Author: peter_firmstone Date: Sun Oct 23 06:30:00 2011 New Revision: 1187847
URL: http://svn.apache.org/viewvc?rev=1187847&view=rev Log: Add comments Modified: river/jtsk/skunk/peterConcurrentPolicy/src/net/jini/security/policy/DynamicPolicyProvider.java Modified: river/jtsk/skunk/peterConcurrentPolicy/src/net/jini/security/policy/DynamicPolicyProvider.java URL: http://svn.apache.org/viewvc/river/jtsk/skunk/peterConcurrentPolicy/src/net/jini/security/policy/DynamicPolicyProvider.java?rev=1187847&r1=1187846&r2=1187847&view=diff ============================================================================== --- river/jtsk/skunk/peterConcurrentPolicy/src/net/jini/security/policy/DynamicPolicyProvider.java (original) +++ river/jtsk/skunk/peterConcurrentPolicy/src/net/jini/security/policy/DynamicPolicyProvider.java Sun Oct 23 06:30:00 2011 @@ -455,6 +455,12 @@ Put the policy providers and all referen * is called. PolicyPermission doesn't imply anything, it * implements toString() for debugging purposes, it's just a * container. + * + * I've since found PolicyPermission's to be unnecessary, the + * ProtectionDomain only merges the permissions for toString() + * not policy decisions. + * + * TODO: Remove PolicyPermission's */ PermissionCollection pc = basePolicy.getPermissions(domain); PermissionCollection permissions = new Permissions();
