Author: angela
Date: Fri Mar 22 14:44:55 2019
New Revision: 1856059

URL: http://svn.apache.org/viewvc?rev=1856059&view=rev
Log:
OAK-8160 : Missing notnull/nullable annotations with GroupPrincipals

Modified:
    
jackrabbit/oak/trunk/oak-security-spi/src/main/java/org/apache/jackrabbit/oak/spi/security/principal/GroupPrincipals.java

Modified: 
jackrabbit/oak/trunk/oak-security-spi/src/main/java/org/apache/jackrabbit/oak/spi/security/principal/GroupPrincipals.java
URL: 
http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-security-spi/src/main/java/org/apache/jackrabbit/oak/spi/security/principal/GroupPrincipals.java?rev=1856059&r1=1856058&r2=1856059&view=diff
==============================================================================
--- 
jackrabbit/oak/trunk/oak-security-spi/src/main/java/org/apache/jackrabbit/oak/spi/security/principal/GroupPrincipals.java
 (original)
+++ 
jackrabbit/oak/trunk/oak-security-spi/src/main/java/org/apache/jackrabbit/oak/spi/security/principal/GroupPrincipals.java
 Fri Mar 22 14:44:55 2019
@@ -58,6 +58,7 @@ public final class GroupPrincipals {
      * @param principal the principal whose membership is listed.
      * @return an enumeration of the group members.
      */
+    @NotNull
     public static Enumeration<? extends Principal> members(@NotNull Principal 
principal) {
         if (principal instanceof Group) {
             return ((Group) principal).members();
@@ -84,7 +85,8 @@ public final class GroupPrincipals {
         return false;
     }
 
-    public static Set<Principal> transform(Set<Group> groups) {
+    @NotNull
+    public static Set<Principal> transform(@NotNull Set<Group> groups) {
         ImmutableSet.Builder<Principal> g2 = ImmutableSet.builder();
         for (Group g : groups) {
             g2.add(new GroupPrincipalWrapper(g));
@@ -92,7 +94,8 @@ public final class GroupPrincipals {
         return g2.build();
     }
 
-    public static Enumeration<? extends Principal> transform(Enumeration<? 
extends Principal> members) {
+    @NotNull
+    public static Enumeration<? extends Principal> transform(@NotNull 
Enumeration<? extends Principal> members) {
         Iterator<Principal> m2 = 
Iterators.transform(Iterators.forEnumeration(members), TRANSFORMER);
         return Iterators.asEnumeration(m2);
     }


Reply via email to