Remove unnecessary code in SecurityExpressionRoot

[lee-ext]

Code:

    private Set<String> getAuthoritySet() {
	if (roles == null) {
		roles = new HashSet<>();
		Collection<? extends GrantedAuthority> userAuthorities = authentication
				.getAuthorities();

		if (roleHierarchy != null) {
			userAuthorities = roleHierarchy
					.getReachableGrantedAuthorities(userAuthorities);
		}

		roles = AuthorityUtils.authorityListToSet(userAuthorities);
	}

	return roles;
}

The "roles = new HashSet<>();" is unnecessary and inefficiency.
Hope it can be modified.

[lee-ext]

In line 161.

[eleftherias]

Thank you for the report @LeeHainie.
I agree that roles = new HashSet<>(); is unnecessary.
Are you interested in submitting a PR to remove it?

[lee-ext]

Yes, I'd love to.

[eleftherias]

Closed in 4b4c6e6.