Minor auto-refactor code cleanup on a massive scale

[labkey-jeckels]

Rationale

We can cleanup tens of thousands of warnings across our codebase with IntelliJ's autorefactors. In some cases, this adopts newer, leaner syntax. In others, it simply removes code that's not serving any purpose.

Changes

• Auto-refactors
  • Eliminate redundant toString()
  • Eliminate redundant cast
  • Empty JavaDoc annotations
  • Member variable can be final
  • length() and size() -> isEmpty()
  • Add missing @OverRide
  • Class can be static
  • Remove unused imports
  • toArray() passed array length
  • Remove unnecessary semicolon
  • Explicit type syntax can be replaced by <>
  • Redundant access modifiers
  • Remove redundant initializer
  • "".equals() -> isEmpty()
  • StringBuilder can be replaced by String
  • Fix misordered arguments to assertEquals()
  • StringUtils.defaultString() - Objects.toString()
  • Cast can be replaced with pattern variable
  • Collapse identical catch blocks
  • Type may be primitive
• Manual fixups
  • Delete unused GWT code
  • Improve generics
  • new UnexpectedException() -> UnexpectedException.wrap()

[brentlogan]

@labkey-jeckels, thanks for the heads up. Is this intended for 25.7?

[labkey-jeckels]

@labkey-jeckels, thanks for the heads up. Is this intended for 25.7?

Yes. These changes touch a lot of lines of code, but the changes are very straightforward and mechanical in nature.