From 40c4a2f35698b88b965cb1b10ded86d38c2ed3f3 Mon Sep 17 00:00:00 2001
From: Kohsuke Kawaguchi <kk@kohsuke.org>
Date: Wed, 24 Sep 2014 17:14:41 -0700
Subject: [PATCH] Got rid of the unused constants

---
 core/src/main/java/jenkins/security/Roles.java | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/core/src/main/java/jenkins/security/Roles.java b/core/src/main/java/jenkins/security/Roles.java
index b433bca6e8..cca26873a5 100644
--- a/core/src/main/java/jenkins/security/Roles.java
+++ b/core/src/main/java/jenkins/security/Roles.java
@@ -2,9 +2,6 @@ package jenkins.security;
 
 import org.jenkinsci.remoting.Role;
 
-import java.util.Collection;
-import java.util.Collections;
-
 /**
  * Predefined {@link Role}s in Jenkins.
  *
@@ -15,17 +12,13 @@ import java.util.Collections;
  * not have any role.
  *
  * @author Kohsuke Kawaguchi
- * @since TODO
+ * @since 1.THU
  */
 public class Roles {
-    public static final Role MASTER = new Role("master");
-
     /**
      * Indicates that a callable runs on masters, requested by slaves/CLI/maven/whatever.
      */
-    public static final Collection<Role> FOR_MASTER = Collections.singleton(MASTER);
-
-    public static final Role SLAVE = new Role("slave");
+    public static final Role MASTER = new Role("master");
 
     /**
      * Indicates that a callable is meant to run on slaves.
@@ -35,8 +28,7 @@ public class Roles {
      * the master (as opposed to ones that do not have that information, which gets
      * {@link Role#UNKNOWN})
      */
-    public static final Collection<Role> FOR_SLAVE = Collections.singleton(SLAVE);
+    public static final Role SLAVE = new Role("slave");
 
     private Roles() {}
-
 }