remove some unused code etc

src/generators/Generator.ts

@@ -242,29 +242,24 @@ export default class Generator {
 					if (name === 'event' && isEventHandler) {
 						// noop
 					} else if (contexts.has(name)) {
-						// if (self.constructor.name === 'DomGenerator') { // TODO filthy, temporary hack
-							const contextName = contexts.get(name);
-							if (contextName !== name) {
-								// this is true for 'reserved' names like `state` and `component`,
-								// also destructured contexts
-
-								code.overwrite(
-									node.start,
-									node.start + name.length,
-									contextName,
-									{ storeName: true, contentOnly: false }
-								);
-
-								const destructuredName = contextName.replace(/\[\d+\]/, '');
-								if (destructuredName !== contextName) {
-									// so that hoisting the context works correctly
-									usedContexts.add(destructuredName);
-								}
+						const contextName = contexts.get(name);
+						if (contextName !== name) {
+							// this is true for 'reserved' names like `state` and `component`,
+							// also destructured contexts
+
+							code.overwrite(
+								node.start,
+								node.start + name.length,
+								contextName,
+								{ storeName: true, contentOnly: false }
+							);
+
+							const destructuredName = contextName.replace(/\[\d+\]/, '');
+							if (destructuredName !== contextName) {
+								// so that hoisting the context works correctly
+								usedContexts.add(destructuredName);
 							}
-
-							// TODO filthy, temporary hack
-							// if (!isEventHandler) code.prependRight(node.start, `state.`);
-						// }
+						}
 
 						usedContexts.add(name);
 					} else if (helpers.has(name)) {
@@ -274,8 +269,6 @@ export default class Generator {
 						const alias = self.templateVars.get(`helpers-${name}`);
 						if (alias !== name) code.overwrite(object.start, object.end, alias);
 					} else if (indexes.has(name)) {
-						if (self.constructor.name === 'DomGenerator' && !isEventHandler) code.prependRight(node.start, `state.`);
-
 						const context = indexes.get(name);
 						usedContexts.add(context); // TODO is this right?
 						usedIndexes.add(name);
@@ -381,8 +374,8 @@ export default class Generator {
 		return alias;
 	}
 
-	getUniqueNameMaker(params: string[]) {
-		const localUsedNames = new Set(params);
+	getUniqueNameMaker() {
+		const localUsedNames = new Set();
 
 		function add(name: string) {
 			localUsedNames.add(name);

src/generators/dom/Block.ts

@@ -114,7 +114,7 @@ export default class Block {
 
 		this.aliases = new Map();
 		this.variables = new Map();
-		this.getUniqueName = this.generator.getUniqueNameMaker([]); // TODO this is wrong... we probably don't need this any more
+		this.getUniqueName = this.generator.getUniqueNameMaker();
 
 		this.hasUpdateMethod = false; // determined later
 	}

src/generators/dom/index.ts

@@ -49,8 +49,8 @@ export class DomGenerator extends Generator {
 		this.metaBindings = [];
 	}
 
-	getUniqueNameMaker(params: string[]) {
-		const localUsedNames = new Set(params); // TODO is this ever called with params?
+	getUniqueNameMaker() {
+		const localUsedNames = new Set();
 
 		function add(name: string) {
 			localUsedNames.add(name);

test/js/samples/each-block-changed-check/expected-bundle.js

@@ -291,7 +291,7 @@ function create_each_block(component, state) {
 		c: function create() {
 			div = createElement("div");
 			strong = createElement("strong");
-			text = createText(state.i);
+			text = createText(i);
 			text_1 = createText("\n\n\t\t");
 			span = createElement("span");
 			text_2 = createText(text_2_value);

test/js/samples/each-block-changed-check/expected.js

@@ -91,7 +91,7 @@ function create_each_block(component, state) {
 		c: function create() {
 			div = createElement("div");
 			strong = createElement("strong");
-			text = createText(state.i);
+			text = createText(i);
 			text_1 = createText("\n\n\t\t");
 			span = createElement("span");
 			text_2 = createText(text_2_value);