From d5885baf6b4061105326209c29d8e58fb18e7f69 Mon Sep 17 00:00:00 2001
From: Luke Taylor <luke.taylor@springsource.com>
Date: Sun, 16 Apr 2006 16:00:32 +0000
Subject: [PATCH] Added some comments.

---
 .../ldap/populator/DefaultLdapAuthoritiesPopulator.java    | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/core/src/main/java/org/acegisecurity/providers/ldap/populator/DefaultLdapAuthoritiesPopulator.java b/core/src/main/java/org/acegisecurity/providers/ldap/populator/DefaultLdapAuthoritiesPopulator.java
index 8aede4dc82..c0bc2df549 100644
--- a/core/src/main/java/org/acegisecurity/providers/ldap/populator/DefaultLdapAuthoritiesPopulator.java
+++ b/core/src/main/java/org/acegisecurity/providers/ldap/populator/DefaultLdapAuthoritiesPopulator.java
@@ -156,8 +156,9 @@ public class DefaultLdapAuthoritiesPopulator implements LdapAuthoritiesPopulator
      * Constructor for group search scenarios. <tt>userRoleAttributes</tt> may still be
      * set as a property.
      *
-     * @param initialDirContextFactory
-     * @param groupSearchBase
+     * @param initialDirContextFactory supplies the contexts used to search for user roles.
+     * @param groupSearchBase if this is an empty string the search will be performed from the root DN of the
+     * context factory.
      */
     public DefaultLdapAuthoritiesPopulator(InitialDirContextFactory initialDirContextFactory, String groupSearchBase) {
         Assert.notNull(initialDirContextFactory, "InitialDirContextFactory must not be null");
@@ -214,7 +215,7 @@ public class DefaultLdapAuthoritiesPopulator implements LdapAuthoritiesPopulator
      * Searches for groups the user is a member of.
      *
      * @param userDn the user's distinguished name.
-     * @param userAttributes
+     * @param userAttributes the retrieved user's attributes (unused by default).
      * @return the set of roles obtained from a group membership search, or null if
      *         <tt>groupSearchBase</tt> has been set.
      */