svn commit: r1735049 - in /sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets: ServletResolver.java package-info.java

[kw...@apache.org]

Author: kwin
Date: Tue Mar 15 09:54:32 2016
New Revision: 1735049

URL: http://svn.apache.org/viewvc?rev=1735049&view=rev
Log:
SLING-5605 fix null annotations and javadoc for return types of all ServletResolver methods

Modified:
    sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/ServletResolver.java
    sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/package-info.java

Modified: sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/ServletResolver.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/ServletResolver.java?rev=1735049&r1=1735048&r2=1735049&view=diff
==============================================================================
--- sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/ServletResolver.java (original)
+++ sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/ServletResolver.java Tue Mar 15 09:54:32 2016
@@ -18,6 +18,7 @@
  */
 package org.apache.sling.api.servlets;
 
+import javax.annotation.CheckForNull;
 import javax.annotation.Nonnull;
 import javax.servlet.Servlet;
 
@@ -57,13 +58,13 @@ public interface ServletResolver {
      * @param request The {@link SlingHttpServletRequest} object used to drive
      *            selection of the servlet.
      * @return The servlet whose <code>service</code> method may be called to
-     *         handle the request.
+     *         handle the request or {@code null} if no appropriate servlet was found.
      * @throws org.apache.sling.api.SlingException Is thrown if an error occurrs
      *             while trying to find an appropriate servlet to handle the
      *             request or if no servlet could be resolved to handle the
      *             request.
      */
-    @Nonnull Servlet resolveServlet(@Nonnull SlingHttpServletRequest request);
+    @CheckForNull Servlet resolveServlet(@Nonnull SlingHttpServletRequest request);
 
     /**
      * Resolves a <code>javax.servlet.Servlet</code> whose
@@ -84,14 +85,14 @@ public interface ServletResolver {
      *                   extension. In this case only a script with the
      *                   matching extension is used.
      * @return The servlet whose <code>service</code> method may be called to
-     *         handle the request.
+     *         handle the request or {@code null} if no appropriate servlet was found.
      * @throws org.apache.sling.api.SlingException Is thrown if an error occurrs
      *             while trying to find an appropriate servlet to handle the
      *             request or if no servlet could be resolved to handle the
      *             request.
      * @since 2.1 (Sling API Bundle 2.1.0)
      */
-    @Nonnull Servlet resolveServlet(@Nonnull Resource resource, @Nonnull String scriptName);
+    @CheckForNull Servlet resolveServlet(@Nonnull Resource resource, @Nonnull String scriptName);
 
     /**
      * Resolves a <code>javax.servlet.Servlet</code> whose
@@ -111,13 +112,13 @@ public interface ServletResolver {
      *                   extension. In this case only a script with the
      *                   matching extension is used.
      * @return The servlet whose <code>service</code> method may be called to
-     *         handle the request.
+     *         handle the request or {@code null} if no appropriate servlet was found.
      * @throws org.apache.sling.api.SlingException Is thrown if an error occurrs
      *             while trying to find an appropriate servlet to handle the
      *             request or if no servlet could be resolved to handle the
      *             request.
      * @since 2.1 (Sling API Bundle 2.1.0)
      */
-    @Nonnull Servlet resolveServlet(@Nonnull ResourceResolver resolver, @Nonnull String scriptName);
+    @CheckForNull Servlet resolveServlet(@Nonnull ResourceResolver resolver, @Nonnull String scriptName);
 
 }

Modified: sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/package-info.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/package-info.java?rev=1735049&r1=1735048&r2=1735049&view=diff
==============================================================================
--- sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/package-info.java (original)
+++ sling/trunk/bundles/api/src/main/java/org/apache/sling/api/servlets/package-info.java Tue Mar 15 09:54:32 2016
@@ -17,7 +17,7 @@
  * under the License.
  */
 
-@Version("2.1.1")
+@Version("2.1.2")
 package org.apache.sling.api.servlets;
 
 import aQute.bnd.annotation.Version;