atom / language-yaml

YAML package for Atom
Other
24 stars 39 forks source link

Rework number tokenization according to the spec #103

Closed winstliu closed 6 years ago

winstliu commented 6 years ago

Requirements

Description of the Change

The previous numeric implementation was custom-written and seems to combine YAML's spec with a bit of Java's. This rewrite makes it conform to the official YAML spec at http://www.yaml.org/spec/1.2/spec.html#id2805071. Some changes include:

Alternate Designs

Keep the existing implementation but tweak it to support the additions listed above. I rejected this because it wouldn't conform to YAML.

Benefits

See above additions.

Possible Drawbacks

See above removals.

Applicable Issues

Fixes #102