tbklang / tlang / #781

81%

master: 50%

Pull #55

coveralls-ruby

🧠 Ideation: Seperation-of-Concerns between AST and Instructions - remove symbol mapper (#63)

* DGen

- Removed ODO

* DGen

- Use `targetName` rather

* Simplifier

- Added this, idk

* Moved

* VariableAssignmentInstr

- Made `varName` and `data` private and added getters instead

VariableDeclaration

- Made `varName`  private and added getter instead

* VariableDeclaration

- Made `varType` private and added a getter instead

* Container

- Added new final method

* Pipelines

- Disabed requirements so we can figure out where the failures may be happening

* Revert "Pipelines"

This reverts commit 53f89db9b.

* Pipelines

- Disabed requirements so we can figure out where the failures may be happening

* DGen

- Only apply simplification to `targetName` when dealing with `VariableAssignmentInstr`
- Only apply simplification to `targetName` when dealing with `VariableDeclaration`
- Only apply simplification to `targetName` when dealing with `FetchValueVar`
- Only apply simplification to `targetName` when dealing with `FuncCallInstr`
- Only apply simplification to `targetName` when dealing with `StackArrayIndexInstruction
- `generateSignature(Function)`	now uses just `paramName` (no simplification as hat makes no sense)
- Fixed	multiple instrucmentation codes	in `emitTestingEntrypoint(File,	Module)`

* TypeChecker

- When handling the `StaticVariableDeclaration` dependency node, just use the `getName()`, don't generate a path from the container to the cvariable entity itself
- Also simplified the above code, neatened it up

* Dependency

- Fixed the for loop stuff (this change done already in another branch) #205

* DGen

- When we do ANY AST lookup do so on the un-`simplify()`'d `getTarget()`, however, the emit
should use the simplified version, call	it `emitTarget`
- Don't	do symbol mapping in `generateSignature_Variable(Variable)`

* Test cases

- Fi... (continued)

384 of 423 new or added lines in 13 files covered. (90.78%)

5063 of 6299 relevant lines covered (80.38%)

426.05 hits per line