File MzLibMinecraft/src/main/java/mz/mzlib/minecraft/hook/Metrics.java has 587 non-comment lines of code

Method Metrics::Metrics has 55 lines of code (limit is 50)

Avoid empty catch blocks

Assigning an Object to null is a code smell.  Consider refactoring.

To be compliant to J2EE, a webapp should not use any thread.

Avoid long parameter lists.

Method MetricsBase::MetricsBase has 14 parameters (limit is 8)

To be compliant to J2EE, a webapp should not use any thread.

To be compliant to J2EE, a webapp should not use any thread.

To be compliant to J2EE, a webapp should not use any thread.

To be compliant to J2EE, a webapp should not use any thread.

Avoid assignments in operands

Return an empty collection rather than null.

The String literal "values" appears 5 times in this file; the first occurrence is on line 442

Avoid instantiating new objects inside loops

Avoid using redundant field initializer for 'hasAtLeastOneField'

Avoid appending characters as strings in StringBuffer.append.

Avoid appending characters as strings in StringBuffer.append.

Avoid appending characters as strings in StringBuffer.append.

Assigning an Object to null is a code smell.  Consider refactoring.

Avoid using Literals in Conditional Statements

Avoid using Literals in Conditional Statements

Avoid using Literals in Conditional Statements

Avoid using Literals in Conditional Statements