From ab493454603c82d0a0a6c0c7b2a7e93a465c90f5 Mon Sep 17 00:00:00 2001 From: Peter Johnson Date: Sun, 9 Dec 2018 08:37:09 -0800 Subject: [PATCH] Add missing param javadoc comment and fix argument order inconsistency (#1475) --- .../main/java/edu/wpi/first/wpilibj/command/PIDSubsystem.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/wpilibj/src/main/java/edu/wpi/first/wpilibj/command/PIDSubsystem.java b/wpilibj/src/main/java/edu/wpi/first/wpilibj/command/PIDSubsystem.java index 3c53e008a0..6465eb1cb2 100644 --- a/wpilibj/src/main/java/edu/wpi/first/wpilibj/command/PIDSubsystem.java +++ b/wpilibj/src/main/java/edu/wpi/first/wpilibj/command/PIDSubsystem.java @@ -89,6 +89,7 @@ public abstract class PIDSubsystem extends Subsystem { * @param p the proportional value * @param i the integral value * @param d the derivative value + * @param f the feed forward value * @param period the time (in seconds) between calculations */ @SuppressWarnings("ParameterName") @@ -124,7 +125,7 @@ public abstract class PIDSubsystem extends Subsystem { * @param period the time (in seconds) between calculations */ @SuppressWarnings("ParameterName") - public PIDSubsystem(double p, double i, double d, double period, double f) { + public PIDSubsystem(double p, double i, double d, double f, double period) { m_controller = new PIDController(p, i, d, f, m_source, m_output, period); addChild("PIDController", m_controller); }