Return to Answer

##Huge Point: Not escaping means not useful

##Big Point: Write real tests

##Mid Point: Inconsistent constructors

##Mid Point: Needless interface

##Mid Point: Needless constants

##Minor Point: Whitespace

Huge Point: Not escaping means not useful

Big Point: Write real tests

Mid Point: Inconsistent constructors

Mid Point: Needless interface

Mid Point: Needless constants

Minor Point: Whitespace

Huge Point: Not escaping means not useful

Big Point: Write real tests

Mid Point: Inconsistent constructors

Mid Point: Needless interface

Mid Point: Needless constants

Minor Point: Whitespace

##Huge Point: Not escaping means not useful

##Big Point: Write real tests

##Mid Point: Inconsistent constructors

##Mid Point: Needless interface

##Mid Point: Needless constants

There's also this:

const TAB = " ";

Which is in fact set to four spaces. "\t" is easier and shorter to type.

##Minor Point: Whitespace

It's more to type and doesn't add any clarity, I therefore suggest they aren't of any value.

It's more to type and doesn't add any clarity.