Team2890/allwpilib
4
0
Fork 0
mirror of https://github.com/wpilibsuite/allwpilib synced 2026-07-04 03:11:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

Override getMessage in Exceptions instead of toString (#7002)

Browse Source 
Fixes https://errorprone.info/bugpattern/OverrideThrowableToString
This commit is contained in:
David Vo
2024-08-27 10:32:32 +10:00
committed by GitHub
parent 1daff9193c
commit 0352a60f38
3 changed files with 4 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
cscore/src/main/java/edu/wpi/first/cscore/VideoException.java
View File

@@ -16,9 +16,4 @@ public class VideoException extends RuntimeException {
public VideoException(String msg) {
super(msg);
}
@Override
public String toString() {
return "VideoException [" + super.toString() + "]";
}
}

4
wpiutil/src/main/java/edu/wpi/first/util/struct/BadSchemaException.java
View File

@@ -73,7 +73,7 @@ public class BadSchemaException extends Exception {
}
@Override
public String toString() {
return m_field.isEmpty() ? getMessage() : "field " + m_field + ": " + getMessage();
public String getMessage() {
return m_field.isEmpty() ? super.getMessage() : "field " + m_field + ": " + super.getMessage();
}
}

4
wpiutil/src/main/java/edu/wpi/first/util/struct/parser/ParseException.java
View File

@@ -53,7 +53,7 @@ public class ParseException extends Exception {
}
@Override
public String toString() {
return m_pos + ": " + getMessage();
public String getMessage() {
return m_pos + ": " + super.getMessage();
}
}