-
Notifications
You must be signed in to change notification settings - Fork 0
Added error handler #27
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great, small changes (http bad) and also want to see how this acts on the actual robot.
@@ -52,8 +53,12 @@ public Robot(HardwareMap hardwareMap, Telemetry telemetry, LinearOpMode linearOp | |||
this.telemetryDump = new TelemetryDump(telemetry); | |||
fileDump = new FileDump(); | |||
|
|||
Thread.UncaughtExceptionHandler exceptionHandler = new KuriousErrorHandler(linearOpMode); | |||
https://docs.oracle.com/javase/1.5.0/docs/api/java/lang/Thread.html#setDefaultUncaughtExceptionHandler(java.lang.Thread.UncaughtExceptionHandler) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bad please fix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it compiles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
very powerful
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
quirk
@@ -52,8 +53,12 @@ public Robot(HardwareMap hardwareMap, Telemetry telemetry, LinearOpMode linearOp | |||
this.telemetryDump = new TelemetryDump(telemetry); | |||
fileDump = new FileDump(); | |||
|
|||
Thread.UncaughtExceptionHandler exceptionHandler = new KuriousErrorHandler(linearOpMode); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you add all exception handlers in a separate function for cleanliness :)
Before issuing a pull request, please see the contributing page.