In the RoleManager API,

Collection findRoleTypes(IdentitySearchControl[] controls) throws IdentityException;

If users pass a 'null' for this method, it will return all of RoleTypes, my question is that is it better that we have a method like:

getAllRoleTypes() to return all of available role types, instead of passing a null parameter in the API level.

The implementation could be resuing as what we have now, as at first sight, I was not sure how can I get all available role types.

Thoughts?