Misc minor checkstyle fixes

Signed-off-by: Greg Wilkins <gregw@webtide.com>
This commit is contained in:
Greg Wilkins 2019-06-25 11:13:11 +02:00
parent 0781e0af3d
commit a42ad99871
8 changed files with 11 additions and 9 deletions

View File

@ -1285,8 +1285,8 @@ public class HttpParser
_string.setLength(0); _string.setLength(0);
_string.append(t.getChar()); _string.append(t.getChar());
_length = 1; _length = 1;
break;
} }
break;
default: default:
throw new IllegalCharacterException(_state, t, buffer); throw new IllegalCharacterException(_state, t, buffer);

View File

@ -25,6 +25,7 @@ import java.util.Queue;
import java.util.concurrent.CompletableFuture; import java.util.concurrent.CompletableFuture;
import java.util.concurrent.ConcurrentLinkedQueue; import java.util.concurrent.ConcurrentLinkedQueue;
import org.eclipse.jetty.util.log.Log;
import org.eclipse.jetty.util.thread.ExecutionStrategy.Producer; import org.eclipse.jetty.util.thread.ExecutionStrategy.Producer;
import org.eclipse.jetty.util.thread.Invocable; import org.eclipse.jetty.util.thread.Invocable;
import org.openjdk.jmh.infra.Blackhole; import org.openjdk.jmh.infra.Blackhole;
@ -124,6 +125,7 @@ public class TestConnection implements Producer
} }
catch (InterruptedException e) catch (InterruptedException e)
{ {
Log.getLogger(TestConnection.class).ignore(e);
} }
} }
else else

View File

@ -81,7 +81,6 @@ public class localContextRoot implements Context
} }
} }
/* /*
* Root has to use the localContextRoot's env for all operations. * Root has to use the localContextRoot's env for all operations.
* So, if createSubcontext in the root, use the env of the localContextRoot. * So, if createSubcontext in the root, use the env of the localContextRoot.

View File

@ -837,6 +837,7 @@ public class JSON
case '\r': case '\r':
case '\n': case '\n':
commentState = 0; commentState = 0;
break;
default: default:
break; break;
} }

View File

@ -759,9 +759,6 @@ public class BlockingArrayQueue<E> extends AbstractList<E> implements BlockingQu
} }
} }
/* Additional methods */
/** /**
* @return the current capacity of this queue * @return the current capacity of this queue
*/ */
@ -786,9 +783,6 @@ public class BlockingArrayQueue<E> extends AbstractList<E> implements BlockingQu
return _maxCapacity; return _maxCapacity;
} }
/* Implementation methods */
private boolean grow() private boolean grow()
{ {
if (_growCapacity <= 0) if (_growCapacity <= 0)

View File

@ -288,8 +288,9 @@ public abstract class IteratingCallback implements Callback
} }
default: default:
throw new IllegalStateException(String.format("%s[action=%s]", this, action)); break;
} }
throw new IllegalStateException(String.format("%s[action=%s]", this, action));
} }
case CALLED: case CALLED:

View File

@ -117,6 +117,7 @@ public class MultiException extends Exception
throw (Error)th; throw (Error)th;
if (th instanceof Exception) if (th instanceof Exception)
throw (Exception)th; throw (Exception)th;
throw new MultiException(nested);
default: default:
throw new MultiException(nested); throw new MultiException(nested);
} }

View File

@ -56,6 +56,8 @@ import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpServletResponseWrapper; import javax.servlet.http.HttpServletResponseWrapper;
import javax.servlet.http.Part; import javax.servlet.http.Part;
import org.eclipse.jetty.util.log.Log;
/** /**
* Dump Servlet Request. * Dump Servlet Request.
*/ */
@ -344,10 +346,12 @@ public class Dump extends HttpServlet
} }
catch (IOException e2) catch (IOException e2)
{ {
Log.getLogger(Dump.class).ignore(e2);
} }
} }
catch (IOException e) catch (IOException e)
{ {
Log.getLogger(Dump.class).ignore(e);
} }
return; return;
} }