From 5cf4c16f5b2d3ae0ba49d928f4285e400d05497a Mon Sep 17 00:00:00 2001 From: Peter Johnson Date: Sun, 26 Jul 2020 17:06:17 -0700 Subject: [PATCH] [wpilibj] Suppress serialVersionUID warnings (#2618) These are all classes (e.g. Exception classes) which will never be serialized. --- .../main/java/edu/wpi/first/hal/util/AllocationException.java | 3 ++- .../main/java/edu/wpi/first/hal/util/BoundaryException.java | 3 ++- .../edu/wpi/first/hal/util/CheckedAllocationException.java | 3 ++- .../main/java/edu/wpi/first/hal/util/HalHandleException.java | 3 ++- .../java/edu/wpi/first/hal/util/UncleanStatusException.java | 3 ++- .../first/wpilibj/command/IllegalUseOfCommandException.java | 3 ++- wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTrigger.java | 3 ++- .../main/java/edu/wpi/first/wpilibj/AnalogTriggerOutput.java | 3 ++- wpilibj/src/main/java/edu/wpi/first/wpilibj/Relay.java | 3 ++- .../java/edu/wpi/first/wpilibj/spline/SplineParameterizer.java | 1 + .../edu/wpi/first/wpilibj/trajectory/TrajectoryGenerator.java | 1 + .../wpi/first/wpilibj/trajectory/TrajectoryParameterizer.java | 1 + 12 files changed, 21 insertions(+), 9 deletions(-) diff --git a/hal/src/main/java/edu/wpi/first/hal/util/AllocationException.java b/hal/src/main/java/edu/wpi/first/hal/util/AllocationException.java index 6f789d8640..42e20592d8 100644 --- a/hal/src/main/java/edu/wpi/first/hal/util/AllocationException.java +++ b/hal/src/main/java/edu/wpi/first/hal/util/AllocationException.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2018 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -10,6 +10,7 @@ package edu.wpi.first.hal.util; /** * Exception indicating that the resource is already allocated. */ +@SuppressWarnings("serial") public class AllocationException extends RuntimeException { /** * Create a new AllocationException. diff --git a/hal/src/main/java/edu/wpi/first/hal/util/BoundaryException.java b/hal/src/main/java/edu/wpi/first/hal/util/BoundaryException.java index 8f21e60417..8fef4c972f 100644 --- a/hal/src/main/java/edu/wpi/first/hal/util/BoundaryException.java +++ b/hal/src/main/java/edu/wpi/first/hal/util/BoundaryException.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2018 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -10,6 +10,7 @@ package edu.wpi.first.hal.util; /** * This exception represents an error in which a lower limit was set as higher than an upper limit. */ +@SuppressWarnings("serial") public class BoundaryException extends RuntimeException { /** * Create a new exception with the given message. diff --git a/hal/src/main/java/edu/wpi/first/hal/util/CheckedAllocationException.java b/hal/src/main/java/edu/wpi/first/hal/util/CheckedAllocationException.java index f17e381ffd..d9a8a806bd 100644 --- a/hal/src/main/java/edu/wpi/first/hal/util/CheckedAllocationException.java +++ b/hal/src/main/java/edu/wpi/first/hal/util/CheckedAllocationException.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2018 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -11,6 +11,7 @@ package edu.wpi.first.hal.util; * Exception indicating that the resource is already allocated This is meant to be thrown by the * resource class. */ +@SuppressWarnings("serial") public class CheckedAllocationException extends Exception { /** * Create a new CheckedAllocationException. diff --git a/hal/src/main/java/edu/wpi/first/hal/util/HalHandleException.java b/hal/src/main/java/edu/wpi/first/hal/util/HalHandleException.java index 874b07ab45..2a7dc9caf5 100644 --- a/hal/src/main/java/edu/wpi/first/hal/util/HalHandleException.java +++ b/hal/src/main/java/edu/wpi/first/hal/util/HalHandleException.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2018 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -10,6 +10,7 @@ package edu.wpi.first.hal.util; /** * Exception indicating that an error has occured with a HAL Handle. */ +@SuppressWarnings("serial") public class HalHandleException extends RuntimeException { /** * Create a new HalHandleException. diff --git a/hal/src/main/java/edu/wpi/first/hal/util/UncleanStatusException.java b/hal/src/main/java/edu/wpi/first/hal/util/UncleanStatusException.java index 3a3e217d02..3d6ab36b7f 100644 --- a/hal/src/main/java/edu/wpi/first/hal/util/UncleanStatusException.java +++ b/hal/src/main/java/edu/wpi/first/hal/util/UncleanStatusException.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2018 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -10,6 +10,7 @@ package edu.wpi.first.hal.util; /** * Exception for bad status codes from the chip object. */ +@SuppressWarnings("serial") public final class UncleanStatusException extends IllegalStateException { private final int m_statusCode; diff --git a/wpilibOldCommands/src/main/java/edu/wpi/first/wpilibj/command/IllegalUseOfCommandException.java b/wpilibOldCommands/src/main/java/edu/wpi/first/wpilibj/command/IllegalUseOfCommandException.java index 1053f55bbd..4ed7459a5a 100644 --- a/wpilibOldCommands/src/main/java/edu/wpi/first/wpilibj/command/IllegalUseOfCommandException.java +++ b/wpilibOldCommands/src/main/java/edu/wpi/first/wpilibj/command/IllegalUseOfCommandException.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2019 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -18,6 +18,7 @@ package edu.wpi.first.wpilibj.command; * it is put into multiple command groups, it is started from outside its command group, or it adds * a new child.

*/ +@SuppressWarnings("serial") public class IllegalUseOfCommandException extends RuntimeException { /** * Instantiates an {@link IllegalUseOfCommandException}. diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTrigger.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTrigger.java index f7f98bde71..282a3462e4 100644 --- a/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTrigger.java +++ b/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTrigger.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2019 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -23,6 +23,7 @@ public class AnalogTrigger implements Sendable, AutoCloseable { /** * Exceptions dealing with improper operation of the Analog trigger. */ + @SuppressWarnings("serial") public static class AnalogTriggerException extends RuntimeException { /** * Create a new exception with the given message. diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTriggerOutput.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTriggerOutput.java index 7635196662..f9fe9509ba 100644 --- a/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTriggerOutput.java +++ b/wpilibj/src/main/java/edu/wpi/first/wpilibj/AnalogTriggerOutput.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2019 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -44,6 +44,7 @@ public class AnalogTriggerOutput extends DigitalSource implements Sendable { /** * Exceptions dealing with improper operation of the Analog trigger output. */ + @SuppressWarnings("serial") public static class AnalogTriggerOutputException extends RuntimeException { /** * Create a new exception with the given message. diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/Relay.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/Relay.java index fde4ef16c4..f379de4be0 100644 --- a/wpilibj/src/main/java/edu/wpi/first/wpilibj/Relay.java +++ b/wpilibj/src/main/java/edu/wpi/first/wpilibj/Relay.java @@ -1,5 +1,5 @@ /*----------------------------------------------------------------------------*/ -/* Copyright (c) 2008-2019 FIRST. All Rights Reserved. */ +/* Copyright (c) 2008-2020 FIRST. All Rights Reserved. */ /* Open Source Software - may be modified and shared by FRC teams. The code */ /* must be accompanied by the FIRST BSD license file in the root directory of */ /* the project. */ @@ -33,6 +33,7 @@ public class Relay extends MotorSafety implements Sendable, AutoCloseable { * This class represents errors in trying to set relay values contradictory to the direction to * which the relay is set. */ + @SuppressWarnings("serial") public static class InvalidValueException extends RuntimeException { /** * Create a new exception with the given message. diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/spline/SplineParameterizer.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/spline/SplineParameterizer.java index 4b7bf95003..15852144a7 100644 --- a/wpilibj/src/main/java/edu/wpi/first/wpilibj/spline/SplineParameterizer.java +++ b/wpilibj/src/main/java/edu/wpi/first/wpilibj/spline/SplineParameterizer.java @@ -62,6 +62,7 @@ public final class SplineParameterizer { } } + @SuppressWarnings("serial") public static class MalformedSplineException extends RuntimeException { /** * Create a new exception with the given message. diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryGenerator.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryGenerator.java index a036de100e..cc1df38c80 100644 --- a/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryGenerator.java +++ b/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryGenerator.java @@ -232,6 +232,7 @@ public final class TrajectoryGenerator { } // Work around type erasure signatures + @SuppressWarnings("serial") public static class ControlVectorList extends ArrayList { public ControlVectorList(int initialCapacity) { super(initialCapacity); diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryParameterizer.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryParameterizer.java index e2bda033fa..3b1d2aff23 100644 --- a/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryParameterizer.java +++ b/wpilibj/src/main/java/edu/wpi/first/wpilibj/trajectory/TrajectoryParameterizer.java @@ -309,6 +309,7 @@ public final class TrajectoryParameterizer { } } + @SuppressWarnings("serial") public static class TrajectoryGenerationException extends RuntimeException { public TrajectoryGenerationException(String message) { super(message);