[java-opensaml COMMIT] in /trunk: opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/BaseContext.jav...

[noreply at shibboleth.net]

Author: scantor
Date: Fri Dec 21 23:25:43 2012
New Revision: 3102

URL: http://svn.shibboleth.net/view/java-opensaml?rev=3102&view=rev
Log:
Nits, annotations, adjust MessageHandler to match other APIs.

Added:
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/decoder/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/decoder/servlet/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/encoder/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/encoder/servlet/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/error/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/error/servlet/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/package-info.java   (with props)
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/package-info.java   (with props)
    trunk/opensaml-messaging-impl/src/main/java/org/opensaml/messaging/handler/impl/package-info.java   (with props)
Modified:
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/BaseContext.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/MessageContextEvaluatingFunctor.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/MessageContextMutatingFunctor.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/navigate/ChildContextLookup.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/navigate/RootContextLookup.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/decoder/MessageDecoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/decoder/servlet/AbstractHttpServletRequestMessageDecoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/decoder/servlet/BaseHttpServletRequestXmlMessageDecoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/decoder/servlet/HttpServletRequestMessageDecoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/encoder/AbstractMessageEncoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/encoder/servlet/AbstractHttpServletResponseMessageEncoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/encoder/servlet/BaseHttpServletResponseXmlMessageEncoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/encoder/servlet/HttpServletResponseMessageEncoder.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/error/PipelineMessageErrorHandler.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/error/TypedMessageErrorHandler.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/error/servlet/AbstractHttpServletMessageErrorHandler.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/error/servlet/HttpServletMessageErrorHandler.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/AbstractMessageHandler.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/MessageHandler.java
    trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/handler/MessageHandlerChain.java
    trunk/opensaml-messaging-impl/src/main/java/org/opensaml/messaging/handler/impl/BasicMessageHandlerChain.java

Modified: trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/BaseContext.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/BaseContext.java?rev=3102&r1=3101&r2=3102&view=diff
==============================================================================
--- trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/BaseContext.java (original)
+++ trunk/opensaml-messaging-api/src/main/java/org/opensaml/messaging/context/BaseContext.java Fri Dec 21 23:25:43 2012
@@ -21,6 +21,8 @@
 import java.lang.reflect.InvocationTargetException;
 import java.util.Iterator;
 import java.util.UUID;
+
+import javax.annotation.Nonnull;
 
 import net.shibboleth.utilities.java.support.collection.ClassIndexedSet;
 import net.shibboleth.utilities.java.support.component.IdentifiableComponent;
@@ -95,8 +97,8 @@
      * 
      * @param newId the new context id
      */
-    protected void setId(final String newId) {
-        id = Constraint.isNotNull(StringSupport.trimOrNull(newId), "Context ID can not be null or empty");
+    protected void setId(@Nonnull final String newId) {
+        id = Constraint.isNotNull(StringSupport.trimOrNull(newId), "Context ID cannot be null or empty");
     }
     
     /**
@@ -133,7 +135,7 @@

[... 811 lines stripped ...]