[stapler] 07/07: Remove use of findbugs

[James Downing Page]

This is an automated email from the git hooks/post-receive script.

jamespage pushed a commit to branch patch-queue/master
in repository stapler.

commit 039c78fec292ce93085ffe6fb3e6be7a52b8e64f
Author: James Page <james.page at ubuntu.com>
Date:   Mon Aug 12 09:45:41 2013 +0100

    Remove use of findbugs
    
    findbugs is not yet packaged so drop references from codebase.
---
 core/pom.xml                                                |    6 ------
 core/src/main/java/org/kohsuke/stapler/HttpRedirect.java    |    5 ++---
 core/src/main/java/org/kohsuke/stapler/ResponseImpl.java    |    9 ++++-----
 core/src/main/java/org/kohsuke/stapler/StaplerResponse.java |    5 ++---
 4 files changed, 8 insertions(+), 17 deletions(-)

diff --git a/core/pom.xml b/core/pom.xml
index 957c1c0..cf06f68 100644
--- a/core/pom.xml
+++ b/core/pom.xml
@@ -128,12 +128,6 @@
       <version>1.0.0</version>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>com.github.stephenc.findbugs</groupId>
-      <artifactId>findbugs-annotations</artifactId>
-      <version>1.3.9-1</version>
-      <scope>provided</scope>
-    </dependency>
   </dependencies>
 
   <profiles>
diff --git a/core/src/main/java/org/kohsuke/stapler/HttpRedirect.java b/core/src/main/java/org/kohsuke/stapler/HttpRedirect.java
index 8d2724a..c80b222 100644
--- a/core/src/main/java/org/kohsuke/stapler/HttpRedirect.java
+++ b/core/src/main/java/org/kohsuke/stapler/HttpRedirect.java
@@ -23,7 +23,6 @@
 
 package org.kohsuke.stapler;
 
-import edu.umd.cs.findbugs.annotations.NonNull;
 import javax.servlet.ServletException;
 import java.io.IOException;
 
@@ -39,11 +38,11 @@ public final class HttpRedirect extends RuntimeException implements HttpResponse
     private final int statusCode;
     private final String url;
 
-    public HttpRedirect(@NonNull String url) {
+    public HttpRedirect(String url) {
         this(SC_MOVED_TEMPORARILY,url);
     }
 
-    public HttpRedirect(int statusCode, @NonNull String url) {
+    public HttpRedirect(int statusCode, String url) {
         this.statusCode = statusCode;
         if (url == null) {
             throw new NullPointerException();
diff --git a/core/src/main/java/org/kohsuke/stapler/ResponseImpl.java b/core/src/main/java/org/kohsuke/stapler/ResponseImpl.java
index 3cc12d2..884cb9f 100644
--- a/core/src/main/java/org/kohsuke/stapler/ResponseImpl.java
+++ b/core/src/main/java/org/kohsuke/stapler/ResponseImpl.java
@@ -23,7 +23,6 @@
 
 package org.kohsuke.stapler;
 
-import edu.umd.cs.findbugs.annotations.NonNull;
 import org.kohsuke.stapler.compression.CompressionFilter;
 import org.kohsuke.stapler.compression.FilterServletOutputStream;
 import org.kohsuke.stapler.export.ExportConfig;
@@ -125,7 +124,7 @@ public class ResponseImpl extends HttpServletResponseWrapper implements StaplerR
     }
 
     @Override
-    public void sendRedirect(@NonNull String url) throws IOException {
+    public void sendRedirect(String url) throws IOException {
         // WebSphere doesn't apparently handle relative URLs, so
         // to be safe, always resolve relative URLs to absolute URLs by ourselves.
         // see http://www.nabble.com/Hudson%3A-1.262%3A-Broken-link-using-update-manager-to21067157.html
@@ -143,13 +142,13 @@ public class ResponseImpl extends HttpServletResponseWrapper implements StaplerR
         super.sendRedirect(base);
     }
 
-    public void sendRedirect2(@NonNull String url) throws IOException {
+    public void sendRedirect2(String url) throws IOException {
         // Tomcat doesn't encode URL (servlet spec isn't very clear on it)
         // so do the encoding by ourselves
         sendRedirect(encode(url));
     }
 
-    public void sendRedirect(int statusCode, @NonNull String url) throws IOException {
+    public void sendRedirect(int statusCode, String url) throws IOException {
         if (statusCode==SC_MOVED_TEMPORARILY) {
             sendRedirect(url);  // to be safe, let the servlet container handles this default case
             return;
@@ -338,7 +337,7 @@ public class ResponseImpl extends HttpServletResponseWrapper implements StaplerR
     /**
      * Escapes non-ASCII characters.
      */
-    public static @NonNull String encode(@NonNull String s) {
+    public static String encode(String s) {
         try {
             boolean escaped = false;
 
diff --git a/core/src/main/java/org/kohsuke/stapler/StaplerResponse.java b/core/src/main/java/org/kohsuke/stapler/StaplerResponse.java
index ff61a67..a29f411 100644
--- a/core/src/main/java/org/kohsuke/stapler/StaplerResponse.java
+++ b/core/src/main/java/org/kohsuke/stapler/StaplerResponse.java
@@ -23,7 +23,6 @@
 
 package org.kohsuke.stapler;
 
-import edu.umd.cs.findbugs.annotations.NonNull;
 import org.kohsuke.stapler.export.Flavor;
 
 import javax.servlet.ServletException;
@@ -71,12 +70,12 @@ public interface StaplerResponse extends HttpServletResponse {
      * Works like {@link #sendRedirect(String)} except that this method
      * escapes the URL.
      */
-    void sendRedirect2(@NonNull String url) throws IOException;
+    void sendRedirect2(String url) throws IOException;
 
     /**
      * Works like {@link #sendRedirect2(String)} but allows the caller to specify the HTTP status code.
      */
-    void sendRedirect(int statusCore, @NonNull String url) throws IOException;
+    void sendRedirect(int statusCore, String url) throws IOException;
 
     /**
      * Serves a static resource.

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-java/stapler.git