From c1cd281f0be23c5971df5d6a572531c4c73856ac Mon Sep 17 00:00:00 2001
From: Sean Krueger <71362472+skrueger-ftc@users.noreply.github.com>
Date: Mon, 9 Oct 2023 17:45:00 -0500
Subject: [PATCH] Remove outdated try/catch from Gamepad Usage (#397)

Since FTC SDK v9.0.0, gamepad.copy() no longer throws a RobotCoreException.
---
 source/docs/software/tutorials/gamepad.rst | 62 ++++++++++------------
 1 file changed, 28 insertions(+), 34 deletions(-)

diff --git a/source/docs/software/tutorials/gamepad.rst b/source/docs/software/tutorials/gamepad.rst
index 2e72b7dc..caab9dd1 100644
--- a/source/docs/software/tutorials/gamepad.rst
+++ b/source/docs/software/tutorials/gamepad.rst
@@ -90,40 +90,34 @@ In a ``LinearOpMode`` based TeleOp program, storing both current and previous ga
 .. code-block::
 
    public void runOpMode() {
-        // By setting these values to new Gamepad(), they will default to all
-        // boolean values as false and all float values as 0
-        Gamepad currentGamepad1 = new Gamepad();
-        Gamepad currentGamepad2 = new Gamepad();
-
-        Gamepad previousGamepad1 = new Gamepad();
-        Gamepad previousGamepad2 = new Gamepad();
-
-        // other initialization code goes here
-
-        while (opModeIsActive()) {
-            try {
-                // Store the gamepad values from the previous loop iteration in
-                // previousGamepad1/2 to be used in this loop iteration.
-                // This is equivalent to doing this at the end of the previous
-                // loop iteration, as it will run in the same order except for
-                // the first/last iteration of the loop.
-                previousGamepad1.copy(currentGamepad1);
-                previousGamepad2.copy(currentGamepad2);
-
-                // Store the gamepad values from this loop iteration in
-                // currentGamepad1/2 to be used for the entirety of this loop iteration.
-                // This prevents the gamepad values from changing between being
-                // used and stored in previousGamepad1/2.
-                currentGamepad1.copy(gamepad1);
-                currentGamepad2.copy(gamepad2);
-            }
-            catch (RobotCoreException e) {
-                // Swallow the possible exception, it should not happen as
-                // currentGamepad1/2 are being copied from valid Gamepads.
-            }
-
-            // Main teleop loop goes here
-        }
+       // By setting these values to new Gamepad(), they will default to all
+       // boolean values as false and all float values as 0
+       Gamepad currentGamepad1 = new Gamepad();
+       Gamepad currentGamepad2 = new Gamepad();
+
+       Gamepad previousGamepad1 = new Gamepad();
+       Gamepad previousGamepad2 = new Gamepad();
+
+       // other initialization code goes here
+
+       while (opModeIsActive()) {
+           // Store the gamepad values from the previous loop iteration in
+           // previousGamepad1/2 to be used in this loop iteration.
+           // This is equivalent to doing this at the end of the previous
+           // loop iteration, as it will run in the same order except for
+           // the first/last iteration of the loop.
+           previousGamepad1.copy(currentGamepad1);
+           previousGamepad2.copy(currentGamepad2);
+
+           // Store the gamepad values from this loop iteration in
+           // currentGamepad1/2 to be used for the entirety of this loop iteration.
+           // This prevents the gamepad values from changing between being
+           // used and stored in previousGamepad1/2.
+           currentGamepad1.copy(gamepad1);
+           currentGamepad2.copy(gamepad2);
+
+           // Main teleop loop goes here
+       }
     }
 
 Rising Edge Detector