Improved Command-based examples (#906)

Robot.cpp has been split into header-source pairs and the Command-based
examples now define an example subsystem in the Robot class.

.travis.yml

@@ -9,7 +9,7 @@ addons:
       - ubuntu-toolchain-r-test
     packages:
       - g++-6
-      - python3.5
+      - python3.5-dev
 
 before_install:
   - sudo sh -c 'echo "deb http://apt.llvm.org/trusty/ llvm-toolchain-trusty-5.0 main" > /etc/apt/sources.list.d/llvm.list'

hal/src/main/native/athena/Notifier.cpp

@@ -169,13 +169,17 @@ void HAL_CleanNotifier(HAL_NotifierHandle notifierHandle, int32_t* status) {
     // the notifier can call back into our callback, so don't hold the lock
     // here (the atomic fetch_sub will prevent multiple parallel entries
     // into this function)
-    if (notifierAlarm) notifierAlarm->writeEnable(false, status);
-    if (notifierManager) notifierManager->disable(status);
 
-    std::lock_guard<wpi::mutex> lock(notifierMutex);
+    // Cleaning up the manager takes up to a second to complete, so don't do
-    notifierAlarm = nullptr;
+    // that here. Fix it more permanently in 2019...
-    notifierManager = nullptr;
+
-    closestTrigger = UINT64_MAX;
+    // if (notifierAlarm) notifierAlarm->writeEnable(false, status);
+    // if (notifierManager) notifierManager->disable(status);
+
+    // std::lock_guard<wpi::mutex> lock(notifierMutex);
+    // notifierAlarm = nullptr;
+    // notifierManager = nullptr;
+    // closestTrigger = UINT64_MAX;
   }
 }
 

hal/src/main/native/athena/PWM.cpp

@@ -104,6 +104,9 @@ HAL_DigitalHandle HAL_InitializePWMPort(HAL_PortHandle portHandle,
   digitalSystem->writeEnableMXPSpecialFunction(specialFunctions | bitToSet,
                                                status);
 
+  // Defaults to allow an always valid config.
+  HAL_SetPWMConfig(handle, 2.0, 1.501, 1.5, 1.499, 1.0, status);
+
   return handle;
 }
 void HAL_FreePWMPort(HAL_DigitalHandle pwmPortHandle, int32_t* status) {

hal/src/main/native/athena/SerialPort.cpp

@@ -43,6 +43,13 @@ void HAL_InitializeSerialPort(HAL_SerialPort port, int32_t* status) {
   if (*status > 0) *status = 0;
 }
 
+void HAL_InitializeSerialPortDirect(HAL_SerialPort port, const char* portName,
+                                    int32_t* status) {
+  *status = viOpen(resourceManagerHandle, const_cast<char*>(portName), VI_NULL,
+                   VI_NULL, reinterpret_cast<ViSession*>(&portHandles[port]));
+  if (*status > 0) *status = 0;
+}
+
 void HAL_SetSerialBaudRate(HAL_SerialPort port, int32_t baud, int32_t* status) {
   *status = viSetAttribute(portHandles[port], VI_ATTR_ASRL_BAUD, baud);
   if (*status > 0) *status = 0;

hal/src/main/native/include/HAL/SerialPort.h

@@ -21,6 +21,8 @@ extern "C" {
 #endif
 
 void HAL_InitializeSerialPort(HAL_SerialPort port, int32_t* status);
+void HAL_InitializeSerialPortDirect(HAL_SerialPort port, const char* portName,
+                                    int32_t* status);
 void HAL_SetSerialBaudRate(HAL_SerialPort port, int32_t baud, int32_t* status);
 void HAL_SetSerialDataBits(HAL_SerialPort port, int32_t bits, int32_t* status);
 void HAL_SetSerialParity(HAL_SerialPort port, int32_t parity, int32_t* status);

hal/src/main/native/sim/PWM.cpp

@@ -57,6 +57,9 @@ HAL_DigitalHandle HAL_InitializePWMPort(HAL_PortHandle portHandle,
 
   SimPWMData[origChannel].SetInitialized(true);
 
+  // Defaults to allow an always valid config.
+  HAL_SetPWMConfig(handle, 2.0, 1.501, 1.5, 1.499, 1.0, status);
+
   return handle;
 }
 void HAL_FreePWMPort(HAL_DigitalHandle pwmPortHandle, int32_t* status) {

hal/src/main/native/sim/SerialPort.cpp

@@ -16,6 +16,9 @@ void InitializeSerialPort() {}
 extern "C" {
 void HAL_InitializeSerialPort(HAL_SerialPort port, int32_t* status) {}
 
+void HAL_InitializeSerialPortDirect(HAL_SerialPort port, const char* portName,
+                                    int32_t* status) {}
+
 void HAL_SetSerialBaudRate(HAL_SerialPort port, int32_t baud, int32_t* status) {
 }
 

wpilibc/src/main/native/cpp/Drive/RobotDriveBase.cpp

@@ -21,9 +21,11 @@ using namespace frc;
 RobotDriveBase::RobotDriveBase() { m_safetyHelper.SetSafetyEnabled(true); }
 
 /**
- * Change the default value for deadband scaling. The default value is
+ * Sets the deadband applied to the drive inputs (e.g., joystick values).
- * 0.02. Values smaller then the deadband are set to 0, while values
+ *
- * larger then the deadband are scaled from 0.0 to 1.0. See ApplyDeadband().
+ * The default value is 0.02. Inputs smaller than the deadband are set to 0.0
+ * while inputs larger than the deadband are scaled from 0.0 to 1.0. See
+ * ApplyDeadband().
  *
  * @param deadband The deadband to set.
  */

wpilibc/src/main/native/cpp/PWM.cpp

@@ -311,8 +311,8 @@ void PWM::SetZeroLatch() {
 }
 
 void PWM::InitSendable(SendableBuilder& builder) {
-  builder.SetSmartDashboardType("Speed Controller");
+  builder.SetSmartDashboardType("PWM");
   builder.SetSafeState([=]() { SetDisabled(); });
-  builder.AddDoubleProperty("Value", [=]() { return GetSpeed(); },
+  builder.AddDoubleProperty("Value", [=]() { return GetRaw(); },
-                            [=](double value) { SetSpeed(value); });
+                            [=](double value) { SetRaw(value); });
 }

wpilibc/src/main/native/cpp/PWMSpeedController.cpp

@@ -7,6 +7,8 @@
 
 #include "PWMSpeedController.h"
 
+#include "SmartDashboard/SendableBuilder.h"
+
 using namespace frc;
 
 /**
@@ -52,3 +54,10 @@ void PWMSpeedController::StopMotor() { SafePWM::StopMotor(); }
  * @param output Write out the PWM value as was found in the PIDController
  */
 void PWMSpeedController::PIDWrite(double output) { Set(output); }
+
+void PWMSpeedController::InitSendable(SendableBuilder& builder) {
+  builder.SetSmartDashboardType("Speed Controller");
+  builder.SetSafeState([=]() { SetDisabled(); });
+  builder.AddDoubleProperty("Value", [=]() { return GetSpeed(); },
+                            [=](double value) { SetSpeed(value); });
+}

wpilibc/src/main/native/cpp/SerialPort.cpp

@@ -61,6 +61,57 @@ SerialPort::SerialPort(int baudRate, Port port, int dataBits,
   HAL_Report(HALUsageReporting::kResourceType_SerialPort, 0);
 }
 
+/**
+ * Create an instance of a Serial Port class.
+ *
+ * @param baudRate The baud rate to configure the serial port.
+ * @param port     The physical port to use
+ * @param portName The direct port name to use
+ * @param dataBits The number of data bits per transfer.  Valid values are
+ *                 between 5 and 8 bits.
+ * @param parity   Select the type of parity checking to use.
+ * @param stopBits The number of stop bits to use as defined by the enum
+ *                 StopBits.
+ */
+SerialPort::SerialPort(int baudRate, llvm::StringRef portName, Port port,
+                       int dataBits, SerialPort::Parity parity,
+                       SerialPort::StopBits stopBits) {
+  int32_t status = 0;
+
+  m_port = port;
+
+  llvm::SmallVector<char, 64> buf;
+  const char* portNameC = portName.c_str(buf);
+
+  HAL_InitializeSerialPortDirect(static_cast<HAL_SerialPort>(port), portNameC,
+                                 &status);
+  wpi_setErrorWithContext(status, HAL_GetErrorMessage(status));
+  // Don't continue if initialization failed
+  if (status < 0) return;
+  HAL_SetSerialBaudRate(static_cast<HAL_SerialPort>(port), baudRate, &status);
+  wpi_setErrorWithContext(status, HAL_GetErrorMessage(status));
+  HAL_SetSerialDataBits(static_cast<HAL_SerialPort>(port), dataBits, &status);
+  wpi_setErrorWithContext(status, HAL_GetErrorMessage(status));
+  HAL_SetSerialParity(static_cast<HAL_SerialPort>(port), parity, &status);
+  wpi_setErrorWithContext(status, HAL_GetErrorMessage(status));
+  HAL_SetSerialStopBits(static_cast<HAL_SerialPort>(port), stopBits, &status);
+  wpi_setErrorWithContext(status, HAL_GetErrorMessage(status));
+
+  // Set the default timeout to 5 seconds.
+  SetTimeout(5.0);
+
+  // Don't wait until the buffer is full to transmit.
+  SetWriteBufferMode(kFlushOnAccess);
+
+  EnableTermination();
+
+  // viInstallHandler(m_portHandle, VI_EVENT_IO_COMPLETION, ioCompleteHandler,
+  // this);
+  // viEnableEvent(m_portHandle, VI_EVENT_IO_COMPLETION, VI_HNDLR, VI_NULL);
+
+  HAL_Report(HALUsageReporting::kResourceType_SerialPort, 0);
+}
+
 /**
  * Destructor.
  */

wpilibc/src/main/native/cpp/SmartDashboard/SendableChooserBase.cpp

@@ -12,3 +12,8 @@ using namespace frc;
 const char* SendableChooserBase::kDefault = "default";
 const char* SendableChooserBase::kOptions = "options";
 const char* SendableChooserBase::kSelected = "selected";
+
+/**
+ * Instantiates a SendableChooser.
+ */
+SendableChooserBase::SendableChooserBase() : SendableBase(false) {}

wpilibc/src/main/native/include/PWMSpeedController.h

@@ -28,6 +28,7 @@ class PWMSpeedController : public SafePWM, public SpeedController {
 
  protected:
   explicit PWMSpeedController(int channel);
+  void InitSendable(SendableBuilder& builder) override;
 
  private:
   bool m_isInverted = false;

wpilibc/src/main/native/include/SerialPort.h

@@ -10,6 +10,7 @@
 #include <string>
 
 #include <llvm/StringRef.h>
+#include <support/deprecated.h>
 
 #include "ErrorBase.h"
 
@@ -56,6 +57,10 @@ class SerialPort : public ErrorBase {
 
   SerialPort(int baudRate, Port port = kOnboard, int dataBits = 8,
              Parity parity = kParity_None, StopBits stopBits = kStopBits_One);
+  WPI_DEPRECATED("Will be removed for 2019")
+  SerialPort(int baudRate, llvm::StringRef portName, Port port = kOnboard,
+             int dataBits = 8, Parity parity = kParity_None,
+             StopBits stopBits = kStopBits_One);
   ~SerialPort();
 
   SerialPort(const SerialPort&) = delete;

wpilibc/src/main/native/include/SmartDashboard/SendableChooser.inc

@@ -62,7 +62,7 @@ void SendableChooser<T>::AddDefault(llvm::StringRef name, T object) {
 template <class T>
 auto SendableChooser<T>::GetSelected()
     -> decltype(_unwrap_smart_ptr(m_choices[""])) {
-  llvm::StringRef selected = m_defaultChoice;
+  std::string selected = m_defaultChoice;
   if (m_selectedEntry) {
     selected = m_selectedEntry.GetString(m_defaultChoice);
   }

wpilibc/src/main/native/include/SmartDashboard/SendableChooserBase.h

@@ -23,6 +23,7 @@ namespace frc {
  */
 class SendableChooserBase : public SendableBase {
  public:
+  SendableChooserBase();
   ~SendableChooserBase() override = default;
 
  protected:

wpilibcExamples/src/main/cpp/templates/commandbased/Commands/ExampleCommand.cpp

@@ -7,9 +7,11 @@
 
 #include "ExampleCommand.h"
 
+#include "../Robot.h"
+
 ExampleCommand::ExampleCommand() {
 	// Use Requires() here to declare subsystem dependencies
-	// eg. Requires(&Robot::chassis);
+	Requires(&Robot::m_subsystem);
 }
 
 // Called just before this Command runs the first time

wpilibcExamples/src/main/cpp/templates/commandbased/Commands/MyAutoCommand.cpp

@@ -7,9 +7,11 @@
 
 #include "MyAutoCommand.h"
 
+#include "../Robot.h"
+
 MyAutoCommand::MyAutoCommand() {
 	// Use Requires() here to declare subsystem dependencies
-	// eg. Requires(&Robot::chassis);
+	Requires(&Robot::m_subsystem);
 }
 
 // Called just before this Command runs the first time

wpilibcExamples/src/main/cpp/templates/commandbased/Robot.cpp

@@ -5,93 +5,77 @@
 /* the project.                                                               */
 /*----------------------------------------------------------------------------*/
 
-#include <Commands/Command.h>
+#include "Robot.h"
+
 #include <Commands/Scheduler.h>
-#include <LiveWindow/LiveWindow.h>
-#include <SmartDashboard/SendableChooser.h>
 #include <SmartDashboard/SmartDashboard.h>
-#include <TimedRobot.h>
 
-#include "Commands/ExampleCommand.h"
+ExampleSubsystem Robot::m_subsystem;
-#include "Commands/MyAutoCommand.h"
+OI Robot::m_oi;
 
-class Robot : public frc::TimedRobot {
+void Robot::RobotInit() {
-public:
+	m_chooser.AddDefault("Default Auto", &m_defaultAuto);
-	void RobotInit() override {
+	m_chooser.AddObject("My Auto", &m_myAuto);
-		m_chooser.AddDefault("Default Auto", &m_defaultAuto);
+	frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
-		m_chooser.AddObject("My Auto", &m_myAuto);
+}
-		frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
+
+/**
+ * This function is called once each time the robot enters Disabled mode. You
+ * can use it to reset any subsystem information you want to clear when the
+ * robot is disabled.
+ */
+void Robot::DisabledInit() {}
+
+void Robot::DisabledPeriodic() {
+	frc::Scheduler::GetInstance()->Run();
+}
+
+/**
+ * This autonomous (along with the chooser code above) shows how to select
+ * between different autonomous modes using the dashboard. The sendable chooser
+ * code works with the Java SmartDashboard. If you prefer the LabVIEW Dashboard,
+ * remove all of the chooser code and uncomment the GetString code to get the
+ * auto name from the text box below the Gyro.
+ *
+ * You can add additional auto modes by adding additional commands to the
+ * chooser code above (like the commented example) or additional comparisons to
+ * the if-else structure below with additional strings & commands.
+ */
+void Robot::AutonomousInit() {
+	// std::string autoSelected = frc::SmartDashboard::GetString(
+	// 		"Auto Selector", "Default");
+	// if (autoSelected == "My Auto") {
+	// 	m_autonomousCommand = &m_myAuto;
+	// } else {
+	// 	m_autonomousCommand = &m_defaultAuto;
+	// }
+
+	m_autonomousCommand = m_chooser.GetSelected();
+
+	if (m_autonomousCommand != nullptr) {
+		m_autonomousCommand->Start();
 	}
+}
 
-	/**
+void Robot::AutonomousPeriodic() {
-	 * This function is called once each time the robot enters Disabled
+	frc::Scheduler::GetInstance()->Run();
-	 * mode.
+}
-	 * You can use it to reset any subsystem information you want to clear
-	 * when
-	 * the robot is disabled.
-	 */
-	void DisabledInit() override {}
 
-	void DisabledPeriodic() override {
+void Robot::TeleopInit() {
-		frc::Scheduler::GetInstance()->Run();
+	// This makes sure that the autonomous stops running when
+	// teleop starts running. If you want the autonomous to
+	// continue until interrupted by another command, remove
+	// this line or comment it out.
+	if (m_autonomousCommand != nullptr) {
+		m_autonomousCommand->Cancel();
+		m_autonomousCommand = nullptr;
 	}
+}
 
-	/**
+void Robot::TeleopPeriodic() {
-	 * This autonomous (along with the chooser code above) shows how to
+	frc::Scheduler::GetInstance()->Run();
-	 * select
+}
-	 * between different autonomous modes using the dashboard. The sendable
-	 * chooser code works with the Java SmartDashboard. If you prefer the
-	 * LabVIEW Dashboard, remove all of the chooser code and uncomment the
-	 * GetString code to get the auto name from the text box below the Gyro.
-	 *
-	 * You can add additional auto modes by adding additional commands to
-	 * the
-	 * chooser code above (like the commented example) or additional
-	 * comparisons
-	 * to the if-else structure below with additional strings & commands.
-	 */
-	void AutonomousInit() override {
-		std::string autoSelected = frc::SmartDashboard::GetString(
-				"Auto Selector", "Default");
-		if (autoSelected == "My Auto") {
-			m_autonomousCommand = &m_myAuto;
-		} else {
-			m_autonomousCommand = &m_defaultAuto;
-		}
 
-		m_autonomousCommand = m_chooser.GetSelected();
+void Robot::TestPeriodic() {}
-
-		if (m_autonomousCommand != nullptr) {
-			m_autonomousCommand->Start();
-		}
-	}
-
-	void AutonomousPeriodic() override {
-		frc::Scheduler::GetInstance()->Run();
-	}
-
-	void TeleopInit() override {
-		// This makes sure that the autonomous stops running when
-		// teleop starts running. If you want the autonomous to
-		// continue until interrupted by another command, remove
-		// this line or comment it out.
-		if (m_autonomousCommand != nullptr) {
-			m_autonomousCommand->Cancel();
-			m_autonomousCommand = nullptr;
-		}
-	}
-
-	void TeleopPeriodic() override { frc::Scheduler::GetInstance()->Run(); }
-
-	void TestPeriodic() override {}
-
-private:
-	// Have it null by default so that if testing teleop it
-	// doesn't have undefined behavior and potentially crash.
-	frc::Command* m_autonomousCommand = nullptr;
-	ExampleCommand m_defaultAuto;
-	MyAutoCommand m_myAuto;
-	frc::SendableChooser<frc::Command*> m_chooser;
-};
 
 START_ROBOT_CLASS(Robot)

wpilibcExamples/src/main/cpp/templates/commandbased/Robot.h

@@ -0,0 +1,40 @@
+/*----------------------------------------------------------------------------*/
+/* Copyright (c) 2017-2018 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.                                                               */
+/*----------------------------------------------------------------------------*/
+
+#pragma once
+
+#include <Commands/Command.h>
+#include <SmartDashboard/SendableChooser.h>
+#include <TimedRobot.h>
+
+#include "Commands/ExampleCommand.h"
+#include "Commands/MyAutoCommand.h"
+#include "OI.h"
+#include "Subsystems/ExampleSubsystem.h"
+
+class Robot : public frc::TimedRobot {
+public:
+	static ExampleSubsystem m_subsystem;
+	static OI m_oi;
+
+	void RobotInit() override;
+	void DisabledInit() override;
+	void DisabledPeriodic() override;
+	void AutonomousInit() override;
+	void AutonomousPeriodic() override;
+	void TeleopInit() override;
+	void TeleopPeriodic() override;
+	void TestPeriodic() override;
+
+private:
+	// Have it null by default so that if testing teleop it
+	// doesn't have undefined behavior and potentially crash.
+	frc::Command* m_autonomousCommand = nullptr;
+	ExampleCommand m_defaultAuto;
+	MyAutoCommand m_myAuto;
+	frc::SendableChooser<frc::Command*> m_chooser;
+};

wpilibcExamples/src/main/cpp/templates/iterative/Robot.cpp

@@ -5,69 +5,54 @@
 /* the project.                                                               */
 /*----------------------------------------------------------------------------*/
 
-#include <iostream>
+#include "Robot.h"
-#include <string>
+
+#include <iostream>
 
-#include <IterativeRobot.h>
-#include <LiveWindow/LiveWindow.h>
-#include <SmartDashboard/SendableChooser.h>
 #include <SmartDashboard/SmartDashboard.h>
 
-class Robot : public frc::IterativeRobot {
+void Robot::RobotInit() {
-public:
+	m_chooser.AddDefault(kAutoNameDefault, kAutoNameDefault);
-	void RobotInit() {
+	m_chooser.AddObject(kAutoNameCustom, kAutoNameCustom);
-		m_chooser.AddDefault(kAutoNameDefault, kAutoNameDefault);
+	frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
-		m_chooser.AddObject(kAutoNameCustom, kAutoNameCustom);
+}
-		frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
+
+/**
+ * This autonomous (along with the chooser code above) shows how to select
+ * between different autonomous modes using the dashboard. The sendable chooser
+ * code works with the Java SmartDashboard. If you prefer the LabVIEW Dashboard,
+ * remove all of the chooser code and uncomment the GetString line to get the
+ * auto name from the text box below the Gyro.
+ *
+ * You can add additional auto modes by adding additional comparisons to the
+ * if-else structure below with additional strings. If using the SendableChooser
+ * make sure to add them to the chooser code above as well.
+ */
+void Robot::AutonomousInit() {
+	m_autoSelected = m_chooser.GetSelected();
+	// m_autoSelected = SmartDashboard::GetString(
+	// 		"Auto Selector", kAutoNameDefault);
+	std::cout << "Auto selected: " << m_autoSelected << std::endl;
+
+	if (m_autoSelected == kAutoNameCustom) {
+		// Custom Auto goes here
+	} else {
+		// Default Auto goes here
 	}
+}
 
-	/*
+void Robot::AutonomousPeriodic() {
-	 * This autonomous (along with the chooser code above) shows how to
+	if (m_autoSelected == kAutoNameCustom) {
-	 * select
+		// Custom Auto goes here
-	 * between different autonomous modes using the dashboard. The sendable
+	} else {
-	 * chooser code works with the Java SmartDashboard. If you prefer the
+		// Default Auto goes here
-	 * LabVIEW Dashboard, remove all of the chooser code and uncomment the
-	 * GetString line to get the auto name from the text box below the Gyro.
-	 *
-	 * You can add additional auto modes by adding additional comparisons to
-	 * the
-	 * if-else structure below with additional strings. If using the
-	 * SendableChooser make sure to add them to the chooser code above as
-	 * well.
-	 */
-	void AutonomousInit() override {
-		m_autoSelected = m_chooser.GetSelected();
-		// m_autoSelected = SmartDashboard::GetString(
-		// 		"Auto Selector", kAutoNameDefault);
-		std::cout << "Auto selected: " << m_autoSelected << std::endl;
-
-		if (m_autoSelected == kAutoNameCustom) {
-			// Custom Auto goes here
-		} else {
-			// Default Auto goes here
-		}
 	}
+}
 
-	void AutonomousPeriodic() {
+void Robot::TeleopInit() {}
-		if (m_autoSelected == kAutoNameCustom) {
-			// Custom Auto goes here
-		} else {
-			// Default Auto goes here
-		}
-	}
 
-	void TeleopInit() {}
+void Robot::TeleopPeriodic() {}
 
-	void TeleopPeriodic() {}
+void Robot::TestPeriodic() {}
-
-	void TestPeriodic() {}
-
-private:
-	frc::LiveWindow& m_lw = *LiveWindow::GetInstance();
-	frc::SendableChooser<std::string> m_chooser;
-	const std::string kAutoNameDefault = "Default";
-	const std::string kAutoNameCustom = "My Auto";
-	std::string m_autoSelected;
-};
 
 START_ROBOT_CLASS(Robot)

wpilibcExamples/src/main/cpp/templates/iterative/Robot.h

@@ -0,0 +1,29 @@
+/*----------------------------------------------------------------------------*/
+/* Copyright (c) 2017-2018 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.                                                               */
+/*----------------------------------------------------------------------------*/
+
+#pragma once
+
+#include <string>
+
+#include <IterativeRobot.h>
+#include <SmartDashboard/SendableChooser.h>
+
+class Robot : public frc::IterativeRobot {
+public:
+	void RobotInit() override;
+	void AutonomousInit() override;
+	void AutonomousPeriodic() override;
+	void TeleopInit() override;
+	void TeleopPeriodic() override;
+	void TestPeriodic() override;
+
+private:
+	frc::SendableChooser<std::string> m_chooser;
+	const std::string kAutoNameDefault = "Default";
+	const std::string kAutoNameCustom = "My Auto";
+	std::string m_autoSelected;
+};

wpilibcExamples/src/main/cpp/templates/sample/Robot.cpp

@@ -5,121 +5,87 @@
 /* the project.                                                               */
 /*----------------------------------------------------------------------------*/
 
-#include <iostream>
+#include "Robot.h"
-#include <string>
+
+#include <iostream>
 
-#include <Drive/DifferentialDrive.h>
-#include <Joystick.h>
-#include <SampleRobot.h>
-#include <SmartDashboard/SendableChooser.h>
 #include <SmartDashboard/SmartDashboard.h>
-#include <Spark.h>
 #include <Timer.h>
 
+Robot::Robot() {
+	// Note SmartDashboard is not initialized here, wait until
+	// RobotInit to make SmartDashboard calls
+	m_robotDrive.SetExpiration(0.1);
+}
+
+void Robot::RobotInit() {
+	m_chooser.AddDefault(kAutoNameDefault, kAutoNameDefault);
+	m_chooser.AddObject(kAutoNameCustom, kAutoNameCustom);
+	frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
+}
+
 /**
- * This is a demo program showing the use of the DifferentialDrive class.
+ * This autonomous (along with the chooser code above) shows how to select
- * The SampleRobot class is the base of a robot application that will
+ * between different autonomous modes using the dashboard. The sendable chooser
- * automatically call your Autonomous and OperatorControl methods at the right
+ * code works with the Java SmartDashboard. If you prefer the LabVIEW Dashboard,
- * time as controlled by the switches on the driver station or the field
+ * remove all of the chooser code and uncomment the GetString line to get the
- * controls.
+ * auto name from the text box below the Gyro.
  *
- * WARNING: While it may look like a good choice to use for your code if you're
+ * You can add additional auto modes by adding additional comparisons to the
- * inexperienced, don't. Unless you know what you are doing, complex code will
+ * if-else structure below with additional strings. If using the SendableChooser
- * be much more difficult under this system. Use IterativeRobot or Command-Based
+ * make sure to add them to the chooser code above as well.
- * instead if you're new.
  */
-class Robot : public frc::SampleRobot {
+void Robot::Autonomous() {
-public:
+	std::string autoSelected = m_chooser.GetSelected();
-	Robot() {
+	// std::string autoSelected = frc::SmartDashboard::GetString(
-		// Note SmartDashboard is not initialized here, wait until
+	// "Auto Selector", kAutoNameDefault);
-		// RobotInit to make SmartDashboard calls
+	std::cout << "Auto selected: " << autoSelected << std::endl;
-		m_robotDrive.SetExpiration(0.1);
+
+	// MotorSafety improves safety when motors are updated in loops
+	// but is disabled here because motor updates are not looped in
+	// this autonomous mode.
+	m_robotDrive.SetSafetyEnabled(false);
+
+	if (autoSelected == kAutoNameCustom) {
+		// Custom Auto goes here
+		std::cout << "Running custom Autonomous" << std::endl;
+
+		// Spin at half speed for two seconds
+		m_robotDrive.ArcadeDrive(0.0, 0.5);
+		frc::Wait(2.0);
+
+		// Stop robot
+		m_robotDrive.ArcadeDrive(0.0, 0.0);
+	} else {
+		// Default Auto goes here
+		std::cout << "Running default Autonomous" << std::endl;
+
+		// Drive forwards at half speed for two seconds
+		m_robotDrive.ArcadeDrive(-0.5, 0.0);
+		frc::Wait(2.0);
+
+		// Stop robot
+		m_robotDrive.ArcadeDrive(0.0, 0.0);
 	}
+}
 
-	void RobotInit() {
+/**
-		m_chooser.AddDefault(kAutoNameDefault, kAutoNameDefault);
+ * Runs the motors with arcade steering.
-		m_chooser.AddObject(kAutoNameCustom, kAutoNameCustom);
+ */
-		frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
+void Robot::OperatorControl() {
+	m_robotDrive.SetSafetyEnabled(true);
+	while (IsOperatorControl() && IsEnabled()) {
+		// Drive with arcade style (use right stick)
+		m_robotDrive.ArcadeDrive(-m_stick.GetY(), m_stick.GetX());
+
+		// The motors will be updated every 5ms
+		frc::Wait(0.005);
 	}
+}
 
-	/*
+/**
-	 * This autonomous (along with the chooser code above) shows how to
+ * Runs during test mode
-	 * select between different autonomous modes using the dashboard. The
+ */
-	 * sendable chooser code works with the Java SmartDashboard. If you
+void Robot::Test() {}
-	 * prefer the LabVIEW Dashboard, remove all of the chooser code and
-	 * uncomment the GetString line to get the auto name from the text box
-	 * below the Gyro.
-	 *
-	 * You can add additional auto modes by adding additional comparisons to
-	 * the if-else structure below with additional strings. If using the
-	 * SendableChooser make sure to add them to the chooser code above as
-	 * well.
-	 */
-	void Autonomous() {
-		std::string autoSelected = m_chooser.GetSelected();
-		// std::string autoSelected = frc::SmartDashboard::GetString(
-		// "Auto Selector", kAutoNameDefault);
-		std::cout << "Auto selected: " << autoSelected << std::endl;
-
-		// MotorSafety improves safety when motors are updated in loops
-		// but is disabled here because motor updates are not looped in
-		// this autonomous mode.
-		m_robotDrive.SetSafetyEnabled(false);
-
-		if (autoSelected == kAutoNameCustom) {
-			// Custom Auto goes here
-			std::cout << "Running custom Autonomous" << std::endl;
-
-			// Spin at half speed for two seconds
-			m_robotDrive.ArcadeDrive(0.0, 0.5);
-			frc::Wait(2.0);
-
-			// Stop robot
-			m_robotDrive.ArcadeDrive(0.0, 0.0);
-		} else {
-			// Default Auto goes here
-			std::cout << "Running default Autonomous" << std::endl;
-
-			// Drive forwards at half speed for two seconds
-			m_robotDrive.ArcadeDrive(-0.5, 0.0);
-			frc::Wait(2.0);
-
-			// Stop robot
-			m_robotDrive.ArcadeDrive(0.0, 0.0);
-		}
-	}
-
-	/*
-	 * Runs the motors with arcade steering.
-	 */
-	void OperatorControl() override {
-		m_robotDrive.SetSafetyEnabled(true);
-		while (IsOperatorControl() && IsEnabled()) {
-			// Drive with arcade style (use right stick)
-			m_robotDrive.ArcadeDrive(
-					-m_stick.GetY(), m_stick.GetX());
-
-			// The motors will be updated every 5ms
-			frc::Wait(0.005);
-		}
-	}
-
-	/*
-	 * Runs during test mode
-	 */
-	void Test() override {}
-
-private:
-	// Robot drive system
-	frc::Spark m_leftMotor{0};
-	frc::Spark m_rightMotor{1};
-	frc::DifferentialDrive m_robotDrive{m_leftMotor, m_rightMotor};
-
-	frc::Joystick m_stick{0};
-
-	frc::SendableChooser<std::string> m_chooser;
-	const std::string kAutoNameDefault = "Default";
-	const std::string kAutoNameCustom = "My Auto";
-};
 
 START_ROBOT_CLASS(Robot)

wpilibcExamples/src/main/cpp/templates/sample/Robot.h

@@ -0,0 +1,50 @@
+/*----------------------------------------------------------------------------*/
+/* Copyright (c) 2017-2018 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.                                                               */
+/*----------------------------------------------------------------------------*/
+
+#pragma once
+
+#include <string>
+
+#include <Drive/DifferentialDrive.h>
+#include <Joystick.h>
+#include <SampleRobot.h>
+#include <SmartDashboard/SendableChooser.h>
+#include <Spark.h>
+
+/**
+ * This is a demo program showing the use of the DifferentialDrive class.
+ * The SampleRobot class is the base of a robot application that will
+ * automatically call your Autonomous and OperatorControl methods at the right
+ * time as controlled by the switches on the driver station or the field
+ * controls.
+ *
+ * WARNING: While it may look like a good choice to use for your code if you're
+ * inexperienced, don't. Unless you know what you are doing, complex code will
+ * be much more difficult under this system. Use IterativeRobot or Command-Based
+ * instead if you're new.
+ */
+class Robot : public frc::SampleRobot {
+public:
+	Robot();
+
+	void RobotInit() override;
+	void Autonomous() override;
+	void OperatorControl() override;
+	void Test() override;
+
+private:
+	// Robot drive system
+	frc::Spark m_leftMotor{0};
+	frc::Spark m_rightMotor{1};
+	frc::DifferentialDrive m_robotDrive{m_leftMotor, m_rightMotor};
+
+	frc::Joystick m_stick{0};
+
+	frc::SendableChooser<std::string> m_chooser;
+	const std::string kAutoNameDefault = "Default";
+	const std::string kAutoNameCustom = "My Auto";
+};

wpilibcExamples/src/main/cpp/templates/timed/Robot.cpp

@@ -5,68 +5,54 @@
 /* the project.                                                               */
 /*----------------------------------------------------------------------------*/
 
+#include "Robot.h"
+
 #include <iostream>
-#include <string>
 
-#include <LiveWindow/LiveWindow.h>
-#include <SmartDashboard/SendableChooser.h>
 #include <SmartDashboard/SmartDashboard.h>
-#include <TimedRobot.h>
 
-class Robot : public frc::TimedRobot {
+void Robot::RobotInit() {
-public:
+	m_chooser.AddDefault(kAutoNameDefault, kAutoNameDefault);
-	void RobotInit() {
+	m_chooser.AddObject(kAutoNameCustom, kAutoNameCustom);
-		m_chooser.AddDefault(kAutoNameDefault, kAutoNameDefault);
+	frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
-		m_chooser.AddObject(kAutoNameCustom, kAutoNameCustom);
+}
-		frc::SmartDashboard::PutData("Auto Modes", &m_chooser);
+
+/**
+ * This autonomous (along with the chooser code above) shows how to select
+ * between different autonomous modes using the dashboard. The sendable chooser
+ * code works with the Java SmartDashboard. If you prefer the LabVIEW Dashboard,
+ * remove all of the chooser code and uncomment the GetString line to get the
+ * auto name from the text box below the Gyro.
+ *
+ * You can add additional auto modes by adding additional comparisons to the
+ * if-else structure below with additional strings. If using the SendableChooser
+ * make sure to add them to the chooser code above as well.
+ */
+void Robot::AutonomousInit() {
+	m_autoSelected = m_chooser.GetSelected();
+	// m_autoSelected = SmartDashboard::GetString("Auto Selector",
+	//		 kAutoNameDefault);
+	std::cout << "Auto selected: " << m_autoSelected << std::endl;
+
+	if (m_autoSelected == kAutoNameCustom) {
+		// Custom Auto goes here
+	} else {
+		// Default Auto goes here
 	}
+}
 
-	/*
+void Robot::AutonomousPeriodic() {
-	 * This autonomous (along with the chooser code above) shows how to
+	if (m_autoSelected == kAutoNameCustom) {
-	 * select between different autonomous modes using the dashboard. The
+		// Custom Auto goes here
-	 * sendable chooser code works with the Java SmartDashboard. If you
+	} else {
-	 * prefer the LabVIEW Dashboard, remove all of the chooser code and
+		// Default Auto goes here
-	 * uncomment the GetString line to get the auto name from the text box
-	 * below the Gyro.
-	 *
-	 * You can add additional auto modes by adding additional comparisons to
-	 * the if-else structure below with additional strings. If using the
-	 * SendableChooser make sure to add them to the chooser code above as
-	 * well.
-	 */
-	void AutonomousInit() override {
-		m_autoSelected = m_chooser.GetSelected();
-		// m_autoSelected = SmartDashboard::GetString("Auto Selector",
-		//		 kAutoNameDefault);
-		std::cout << "Auto selected: " << m_autoSelected << std::endl;
-
-		if (m_autoSelected == kAutoNameCustom) {
-			// Custom Auto goes here
-		} else {
-			// Default Auto goes here
-		}
 	}
+}
 
-	void AutonomousPeriodic() {
+void Robot::TeleopInit() {}
-		if (m_autoSelected == kAutoNameCustom) {
-			// Custom Auto goes here
-		} else {
-			// Default Auto goes here
-		}
-	}
 
-	void TeleopInit() {}
+void Robot::TeleopPeriodic() {}
 
-	void TeleopPeriodic() {}
+void Robot::TestPeriodic() {}
-
-	void TestPeriodic() {}
-
-private:
-	frc::LiveWindow& m_lw = *LiveWindow::GetInstance();
-	frc::SendableChooser<std::string> m_chooser;
-	const std::string kAutoNameDefault = "Default";
-	const std::string kAutoNameCustom = "My Auto";
-	std::string m_autoSelected;
-};
 
 START_ROBOT_CLASS(Robot)

wpilibcExamples/src/main/cpp/templates/timed/Robot.h

@@ -0,0 +1,29 @@
+/*----------------------------------------------------------------------------*/
+/* Copyright (c) 2017-2018 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.                                                               */
+/*----------------------------------------------------------------------------*/
+
+#pragma once
+
+#include <string>
+
+#include <SmartDashboard/SendableChooser.h>
+#include <TimedRobot.h>
+
+class Robot : public frc::TimedRobot {
+public:
+	void RobotInit() override;
+	void AutonomousInit() override;
+	void AutonomousPeriodic() override;
+	void TeleopInit() override;
+	void TeleopPeriodic() override;
+	void TestPeriodic() override;
+
+private:
+	frc::SendableChooser<std::string> m_chooser;
+	const std::string kAutoNameDefault = "Default";
+	const std::string kAutoNameCustom = "My Auto";
+	std::string m_autoSelected;
+};

wpilibj/src/main/java/edu/wpi/first/wpilibj/ADXRS450_Gyro.java

@@ -80,6 +80,10 @@ public class ADXRS450_Gyro extends GyroBase implements Gyro, PIDSource, Sendable
     setName("ADXRS450_Gyro", port.value);
   }
 
+  public boolean isConnected() {
+    return m_spi != null;
+  }
+
   @Override
   public void calibrate() {
     if (m_spi == null) {
@@ -128,7 +132,9 @@ public class ADXRS450_Gyro extends GyroBase implements Gyro, PIDSource, Sendable
 
   @Override
   public void reset() {
-    m_spi.resetAccumulator();
+    if (m_spi != null) {
+      m_spi.resetAccumulator();
+    }
   }
 
   /**

wpilibj/src/main/java/edu/wpi/first/wpilibj/CameraServer.java

@@ -252,39 +252,43 @@ public class CameraServer {
     }
 
     NetworkTableEntry entry = table.getEntry(name);
-    switch (event.propertyKind) {
+    try {
-      case kBoolean:
+      switch (event.propertyKind) {
-        if (isNew) {
+        case kBoolean:
-          entry.setDefaultBoolean(event.value != 0);
+          if (isNew) {
-        } else {
+            entry.setDefaultBoolean(event.value != 0);
-          entry.setBoolean(event.value != 0);
+          } else {
-        }
+            entry.setBoolean(event.value != 0);
-        break;
+          }
-      case kInteger:
+          break;
-      case kEnum:
+        case kInteger:
-        if (isNew) {
+        case kEnum:
-          entry.setDefaultDouble(event.value);
+          if (isNew) {
-          table.getEntry(infoName + "/min").setDouble(
+            entry.setDefaultDouble(event.value);
-              CameraServerJNI.getPropertyMin(event.propertyHandle));
+            table.getEntry(infoName + "/min").setDouble(
-          table.getEntry(infoName + "/max").setDouble(
+                CameraServerJNI.getPropertyMin(event.propertyHandle));
-              CameraServerJNI.getPropertyMax(event.propertyHandle));
+            table.getEntry(infoName + "/max").setDouble(
-          table.getEntry(infoName + "/step").setDouble(
+                CameraServerJNI.getPropertyMax(event.propertyHandle));
-              CameraServerJNI.getPropertyStep(event.propertyHandle));
+            table.getEntry(infoName + "/step").setDouble(
-          table.getEntry(infoName + "/default").setDouble(
+                CameraServerJNI.getPropertyStep(event.propertyHandle));
-              CameraServerJNI.getPropertyDefault(event.propertyHandle));
+            table.getEntry(infoName + "/default").setDouble(
-        } else {
+                CameraServerJNI.getPropertyDefault(event.propertyHandle));
-          entry.setDouble(event.value);
+          } else {
-        }
+            entry.setDouble(event.value);
-        break;
+          }
-      case kString:
+          break;
-        if (isNew) {
+        case kString:
-          entry.setDefaultString(event.valueStr);
+          if (isNew) {
-        } else {
+            entry.setDefaultString(event.valueStr);
-          entry.setString(event.valueStr);
+          } else {
-        }
+            entry.setString(event.valueStr);
-        break;
+          }
-      default:
+          break;
-        break;
+        default:
+          break;
+      }
+    } catch (VideoException ex) {
+      // ignore
     }
   }
 
@@ -388,8 +392,12 @@ public class CameraServer {
         case kSourcePropertyChoicesUpdated: {
           NetworkTable table = m_tables.get(event.sourceHandle);
           if (table != null) {
-            String[] choices = CameraServerJNI.getEnumPropertyChoices(event.propertyHandle);
+            try {
-            table.getEntry("PropertyInfo/" + event.name + "/choices").setStringArray(choices);
+              String[] choices = CameraServerJNI.getEnumPropertyChoices(event.propertyHandle);
+              table.getEntry("PropertyInfo/" + event.name + "/choices").setStringArray(choices);
+            } catch (VideoException ex) {
+              // ignore
+            }
           }
           break;
         }

wpilibj/src/main/java/edu/wpi/first/wpilibj/PWM.java

@@ -244,8 +244,8 @@ public class PWM extends SendableBase implements Sendable {
 
   @Override
   public void initSendable(SendableBuilder builder) {
-    builder.setSmartDashboardType("Speed Controller");
+    builder.setSmartDashboardType("PWM");
     builder.setSafeState(this::setDisabled);
-    builder.addDoubleProperty("Value", this::getSpeed, this::setSpeed);
+    builder.addDoubleProperty("Value", this::getRaw, value -> setRaw((int) value));
   }
 }

wpilibj/src/main/java/edu/wpi/first/wpilibj/PWMSpeedController.java

@@ -7,6 +7,8 @@
 
 package edu.wpi.first.wpilibj;
 
+import edu.wpi.first.wpilibj.smartdashboard.SendableBuilder;
+
 /**
  * Common base class for all PWM Speed Controllers.
  */
@@ -66,4 +68,11 @@ public abstract class PWMSpeedController extends SafePWM implements SpeedControl
   public void pidWrite(double output) {
     set(output);
   }
+
+  @Override
+  public void initSendable(SendableBuilder builder) {
+    builder.setSmartDashboardType("Speed Controller");
+    builder.setSafeState(this::setDisabled);
+    builder.addDoubleProperty("Value", this::getSpeed, this::setSpeed);
+  }
 }

wpilibj/src/main/java/edu/wpi/first/wpilibj/SerialPort.java

@@ -94,6 +94,42 @@ public class SerialPort {
     }
   }
 
+  /**
+   * Create an instance of a Serial Port class.
+   *
+   * @param baudRate The baud rate to configure the serial port.
+   * @param port     The Serial port to use
+   * @param portName The direct portName to use
+   * @param dataBits The number of data bits per transfer. Valid values are between 5 and 8 bits.
+   * @param parity   Select the type of parity checking to use.
+   * @param stopBits The number of stop bits to use as defined by the enum StopBits.
+   * @deprecated     Will be removed for 2019
+   */
+  @Deprecated
+  public SerialPort(final int baudRate, String portName, Port port, final int dataBits,
+                    Parity parity, StopBits stopBits) {
+    m_port = (byte) port.value;
+
+    SerialPortJNI.serialInitializePortDirect(m_port, portName);
+    SerialPortJNI.serialSetBaudRate(m_port, baudRate);
+    SerialPortJNI.serialSetDataBits(m_port, (byte) dataBits);
+    SerialPortJNI.serialSetParity(m_port, (byte) parity.value);
+    SerialPortJNI.serialSetStopBits(m_port, (byte) stopBits.value);
+
+    // Set the default read buffer size to 1 to return bytes immediately
+    setReadBufferSize(1);
+
+    // Set the default timeout to 5 seconds.
+    setTimeout(5.0);
+
+    // Don't wait until the buffer is full to transmit.
+    setWriteBufferMode(WriteBufferMode.kFlushOnAccess);
+
+    disableTermination();
+
+    HAL.report(tResourceType.kResourceType_SerialPort, 0);
+  }
+
   /**
    * Create an instance of a Serial Port class.
    *

wpilibj/src/main/java/edu/wpi/first/wpilibj/drive/RobotDriveBase.java

@@ -43,9 +43,11 @@ public abstract class RobotDriveBase extends SendableBase implements MotorSafety
   }
 
   /**
-   * Change the default value for deadband scaling. The default value is
+   * Sets the deadband applied to the drive inputs (e.g., joystick values).
-   * {@value #kDefaultDeadband}. Values smaller then the deadband are set to 0, while values
+   *
-   * larger then the deadband are scaled from 0.0 to 1.0. See {@link #applyDeadband}.
+   * <p>The default value is {@value #kDefaultDeadband}. Inputs smaller than the deadband are set to
+   * 0.0 while inputs larger than the deadband are scaled from 0.0 to 1.0. See
+   * {@link #applyDeadband}.
    *
    * @param deadband The deadband to set.
    */

wpilibj/src/main/java/edu/wpi/first/wpilibj/hal/SerialPortJNI.java

@@ -10,6 +10,8 @@ package edu.wpi.first.wpilibj.hal;
 public class SerialPortJNI extends JNIWrapper {
   public static native void serialInitializePort(byte port);
 
+  public static native void serialInitializePortDirect(byte port, String portName);
+
   public static native void serialSetBaudRate(byte port, int baud);
 
   public static native void serialSetDataBits(byte port, byte bits);

wpilibj/src/main/java/edu/wpi/first/wpilibj/smartdashboard/SendableChooser.java

@@ -51,6 +51,7 @@ public class SendableChooser<V> extends SendableBase implements Sendable {
    * Instantiates a {@link SendableChooser}.
    */
   public SendableChooser() {
+    super(false);
   }
 
   /**

wpilibj/src/main/native/cpp/SerialPortJNI.cpp

@@ -45,6 +45,25 @@ Java_edu_wpi_first_wpilibj_hal_SerialPortJNI_serialInitializePort(
   CheckStatusForceThrow(env, status);
 }
 
+/*
+ * Class:     edu_wpi_first_wpilibj_hal_SerialPortJNI
+ * Method:    serialInitializePortDirect
+ * Signature: (BLjava/lang/String;)V
+ */
+JNIEXPORT void JNICALL
+Java_edu_wpi_first_wpilibj_hal_SerialPortJNI_serialInitializePortDirect(
+    JNIEnv* env, jclass, jbyte port, jstring portName) {
+  SERIALJNI_LOG(logDEBUG) << "Calling Serial Initialize Direct";
+  SERIALJNI_LOG(logDEBUG) << "Port = " << (jint)port;
+  JStringRef portNameRef{env, portName};
+  SERIALJNI_LOG(logDEBUG) << "PortName = " << portNameRef.c_str();
+  int32_t status = 0;
+  HAL_InitializeSerialPortDirect(static_cast<HAL_SerialPort>(port),
+                                 portNameRef.c_str(), &status);
+  SERIALJNI_LOG(logDEBUG) << "Status = " << status;
+  CheckStatusForceThrow(env, status);
+}
+
 /*
  * Class:     edu_wpi_first_wpilibj_hal_SerialPortJNI
  * Method:    serialSetBaudRate

wpilibjExamples/src/main/java/edu/wpi/first/wpilibj/templates/commandbased/Robot.java

@@ -23,8 +23,7 @@ import edu.wpi.first.wpilibj.templates.commandbased.subsystems.ExampleSubsystem;
  * project.
  */
 public class Robot extends TimedRobot {
-	public static final ExampleSubsystem kExampleSubsystem
+	public static ExampleSubsystem m_subsystem = new ExampleSubsystem();
-			= new ExampleSubsystem();
 	public static OI m_oi;
 
 	Command m_autonomousCommand;

wpilibjExamples/src/main/java/edu/wpi/first/wpilibj/templates/commandbased/commands/ExampleCommand.java

@@ -16,7 +16,7 @@ import edu.wpi.first.wpilibj.templates.commandbased.Robot;
 public class ExampleCommand extends Command {
 	public ExampleCommand() {
 		// Use requires() here to declare subsystem dependencies
-		requires(Robot.kExampleSubsystem);
+		requires(Robot.m_subsystem);
 	}
 
 	// Called just before this Command runs the first time