Adhere to stated style guide
... or clearly document exceptions.
The project Wiki claims the code loosely follows the Google Java Style Guide. The project should more strictly adhere to the standards in that document or should document the exceptions to the style guide. Of note, the following rules are violated (ordered by appearance in style guide itself):
- 2.3.1 Whitespace characters
- 3.3.3 Ordering and spacing
- 3.4.2 Class member ordering
- 4.1.1. Braces are used where optional
- 4.2 Block indentation: +2 spaces
- 4.4 Column limit: 80 or 100
- Note: While columns never exceed 100, it appears 80 was the intended target and that is exceeded in multiple locations.
- 4.5.2 Indent continuation lines at least +4 spaces
- 5.2 Rules common to all identifiers
- Use of unicode (non-ascii) member names
- 5.2.2 Class names
- Some class names are not nouns, eg. “ClusterToCurves”
- 5.3 Camel case: defined
- Some references like: “anXMLVar” should be “anXmlVar”
- 6.3 Static members: qualified using class
- 7.2 The summary fragment