You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sm...@apache.org on 2015/05/29 22:25:49 UTC

directory-fortress-realm git commit: FC-105 - declare final and name change

Repository: directory-fortress-realm
Updated Branches:
  refs/heads/master 1dceb9d51 -> ceb972060


FC-105 - declare final and name change


Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-realm/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-realm/commit/ceb97206
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-realm/tree/ceb97206
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-realm/diff/ceb97206

Branch: refs/heads/master
Commit: ceb9720609dc9a0731b4cd19e24c6ec067361c27
Parents: 1dceb9d
Author: Shawn McKinney <sm...@apache.org>
Authored: Fri May 29 15:25:45 2015 -0500
Committer: Shawn McKinney <sm...@apache.org>
Committed: Fri May 29 15:25:45 2015 -0500

----------------------------------------------------------------------
 .../org/apache/directory/fortress/realm/GlobalIds.java  | 12 ++++++------
 .../directory/fortress/realm/J2eePolicyMgrFactory.java  |  2 +-
 .../apache/directory/fortress/realm/util/CpUtil.java    |  6 +++---
 3 files changed, 10 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/directory-fortress-realm/blob/ceb97206/impl/src/main/java/org/apache/directory/fortress/realm/GlobalIds.java
----------------------------------------------------------------------
diff --git a/impl/src/main/java/org/apache/directory/fortress/realm/GlobalIds.java b/impl/src/main/java/org/apache/directory/fortress/realm/GlobalIds.java
index fce9858..9d5fcb5 100644
--- a/impl/src/main/java/org/apache/directory/fortress/realm/GlobalIds.java
+++ b/impl/src/main/java/org/apache/directory/fortress/realm/GlobalIds.java
@@ -3,16 +3,16 @@ package org.apache.directory.fortress.realm;
 /**
  * Created by smckinn on 3/12/15.
  */
-public class GlobalIds
+public final class GlobalIds
 {
     // disallow construction
     private GlobalIds()
     {
     }
 
-    public static int CONTEXT_SERIALIZATION_FAILED = 201;
-    public static int CONTEXT_DESERIALIZATION_FAILED_IO = 202;
-    public static int CONTEXT_DESERIALIZATION_FAILED_UNSUPPORTED_ENCODING = 203;
-    public static int CONTEXT_DESERIALIZATION_FAILED_CLASS_NOT_FOUND = 204;
-    public static int SESSION_INITIALIZATION_FAILED = 205;
+    public static final int CONTEXT_SERIALIZATION_FAILED = 201;
+    public static final int CONTEXT_DESERIALIZATION_FAILED_IO = 202;
+    public static final int CONTEXT_DESERIALIZATION_FAILED_UNSUPPORTED_ENCODING = 203;
+    public static final int CONTEXT_DESERIALIZATION_FAILED_CLASS_NOT_FOUND = 204;
+    public static final int SESSION_INITIALIZATION_FAILED = 205;
 }

http://git-wip-us.apache.org/repos/asf/directory-fortress-realm/blob/ceb97206/impl/src/main/java/org/apache/directory/fortress/realm/J2eePolicyMgrFactory.java
----------------------------------------------------------------------
diff --git a/impl/src/main/java/org/apache/directory/fortress/realm/J2eePolicyMgrFactory.java b/impl/src/main/java/org/apache/directory/fortress/realm/J2eePolicyMgrFactory.java
index d72a798..8a2d475 100644
--- a/impl/src/main/java/org/apache/directory/fortress/realm/J2eePolicyMgrFactory.java
+++ b/impl/src/main/java/org/apache/directory/fortress/realm/J2eePolicyMgrFactory.java
@@ -34,7 +34,7 @@ import org.slf4j.LoggerFactory;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-public class J2eePolicyMgrFactory
+public final class J2eePolicyMgrFactory
 {
     // disallow construction
     private J2eePolicyMgrFactory()

http://git-wip-us.apache.org/repos/asf/directory-fortress-realm/blob/ceb97206/proxy/src/main/java/org/apache/directory/fortress/realm/util/CpUtil.java
----------------------------------------------------------------------
diff --git a/proxy/src/main/java/org/apache/directory/fortress/realm/util/CpUtil.java b/proxy/src/main/java/org/apache/directory/fortress/realm/util/CpUtil.java
index 0a254ee..0ae012c 100644
--- a/proxy/src/main/java/org/apache/directory/fortress/realm/util/CpUtil.java
+++ b/proxy/src/main/java/org/apache/directory/fortress/realm/util/CpUtil.java
@@ -31,17 +31,17 @@ import java.util.StringTokenizer;
  *
  * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
  */
-public class CpUtil
+public final class CpUtil
 {
     private static final String CLS_NM = CpUtil.class.getName();
-    private static Logger LOG = Logger.getLogger(CLS_NM);
+    private static final Logger LOG = Logger.getLogger(CLS_NM);
     private static final String PATH_SEPARATOR = "path.separator";
     private static final String FILE_PREFIX = "file:/";
     private static final String UNIX_SLASH = "/";
     private static final String WIN_SLASH = "\\";
     private static final String UNIX_SEPARATOR = ":";
 
-    // disallow construction
+    // disallow outside construction
     private CpUtil()
     {
     }