Bringing back some cleanup from a past PR. Helps make the code more readable and also helps with code quality.
Proposed Changes
In a variety of classes (Configuration, VFXParticle, LootBeamRenderer, ApotheosisCompat) high levels of indentation has been mitigated or removed through the use of guard statements. Spaces have been added between {} and calls to increase visibility. Empty lines added in places to denote a different group of code or just increase readability. Name scheme has been updated in places to consistently use camelCase. As well as just general cleanup. (Removing Unused variables, fields, imports, methods, etc)
In LootBeamRenderer specifically, some nearly identical methods have been merged and parameter names have been updated to be easier to understand. Some comments have had a space added between the // and the actual comment. If statements have been merged together, etc.
Description
Bringing back some cleanup from a past PR. Helps make the code more readable and also helps with code quality.
Proposed Changes
Configuration,VFXParticle,LootBeamRenderer,ApotheosisCompat) high levels of indentation has been mitigated or removed through the use of guard statements. Spaces have been added between {} and calls to increase visibility. Empty lines added in places to denote a different group of code or just increase readability. Name scheme has been updated in places to consistently use camelCase. As well as just general cleanup. (Removing Unused variables, fields, imports, methods, etc)LootBeamRendererspecifically, some nearly identical methods have been merged and parameter names have been updated to be easier to understand. Some comments have had a space added between the // and the actual comment. If statements have been merged together, etc.As of now this PR is: untested