dkubb / sql

100%

master: 100%

LAST BUILD ON BRANCH class-variable-emitter-registry

branch: class-variable-emitter-registry

CHANGE BRANCH

x

Reset

• class-variable-emitter-registry
• add-parent-node
• change-buffer-to-stream
• collapse-select
• insert-emitter
• kill-string-emitter-mutation
• master
• parser
• rename-nodes
• replace-delimited
• update-emitter

Build # 60

Build Type

push

travis-ci

Committed by dkubb

Commit Message

Update registry to use a class variable

* Rather than creating a private method for the emitter registry, a
  class variable can be used to share state between classes in the
  same hierarchy. Normally I would not use class variables for much,
  except this is exactly the use case they were designed for.
* Add Emitter.finalize to hide the class internal state from the
  outside world. Nothing outside of the Emitter class needs to know
  there is a registry object internally, there is no need to expose
  that in a public interface.

Run Details

243 of 243 relevant lines covered (100.0%)

45.35 hits per line